simondlevy / cppsockets Goto Github PK

Thanks for making this nice simple library.

I have an application which needs to receive UDP packets on port 56565, and this gives me issues because I notice that the UdpSocketSocket expects short which cannot represent port numbers that are this "high":

UdpServerSocket(const short port, const uint32_t timeoutMsec=0)

Changing this to something else (e.g. unsigned short or uint16_t does not appear to help; perhaps there is some other limitation I'm not aware of?

Throws a few errors, one of them related to uint32_t not being declared (fixed that by including cstdio in SocketCompat)
Then one relating to an attempt to pass a WCHAR pointer as an argument where an LPCSTR is expected.

Log:

$ g++ ./examples/tcpserver.cpp -I./sockets -o test
In file included from ./sockets/TcpSocket.hpp:11,
from ./sockets/TcpServerSocket.hpp:11,
from ./examples/tcpserver.cpp:10:
./sockets/SocketCompat.hpp:73:28: error: 'uint32_t' has not been declared
73 | void setUdpTimeout(uint32_t msec)
| ^~~~~~~~
./sockets/SocketCompat.hpp: In member function 'void Socket::inetPton(const char*, sockaddr_in&)':
./sockets/SocketCompat.hpp:67:31: error: cannot convert 'WCHAR*' {aka 'wchar_t*'} to 'LPCSTR' {aka
const char*'}
67 | InetPton(AF_INET, wsz, &(saddr_in.sin_addr.s_addr));
| ^~~
| |
| WCHAR* {aka wchar_t*}
In file included from ./sockets/SocketCompat.hpp:17:
C:/msys64/mingw64/include/ws2tcpip.h:407:61: note: initializing argument 2 of 'INT inet_pton(INT,
LPCSTR, PVOID)'
407 | WINSOCK_API_LINKAGE INT WSAAPI InetPtonA(INT Family, LPCSTR pStringBuf, PVOID pAddr);
| ~~~~~~~^~~~~~~~~~