ObjFW  History of src/OFSocket.h of 948a5c25ce6950a5

It seems that neither Windows nor macOS support DDP sockets, but only
the layers on top, while neither Linux nor NetBSD support the layers on
top, but only provide DDP.

It seems that DDP is not meant to be used directly, while Linux and
NetBSD only provide the minimum to use netatalk, which is DDP. ObjFW
would need to use the layers on top on Windows and macOS and implement
those layers itself on Linux and NetBSD. This also means that no code
would be shared between Linux / NetBSD and Windows / macOS, as it's
impossible to have one implementation that works for everything. At this
point, the effort required doesn't make sense. file: [fc6ec6273c] check-in: [0b4f713964] user: js, branch: trunk, size: 9155 [annotate] [blame] [check-ins using] [diff]

While it works for those two protocols, it's a bad precedent as other
protocols have different types for ports. file: [e6bf54de5e] check-in: [a54730b88f] user: js, branch: trunk, size: 9155 [annotate] [blame] [check-ins using] [diff]

struct sockaddr can contain a variable length array, which breaks the
union. file: [0929d955af] check-in: [25adea9a23] user: js, branch: trunk, size: 8480 [annotate] [blame] [check-ins using] [diff]

This is more standard, albeit -[connectToNetwork:node:port:] sounding a
little bit more like you connect to a network rather than a node. file: [e523ef42af] check-in: [ccc6a77c55] user: js, branch: trunk, size: 8749 [annotate] [blame] [check-ins using] [diff]

Windows 10 supports UNIX sockets since Insider Preview Build 17063, but
only SOCK_STREAM. file: [65bba562bd] check-in: [d5373ff70f] user: js, branch: unix-sockets, size: 8733 [annotate] [blame] [check-ins using] [diff]

With TYPEDEF_HIDES_STRUCT set, Doxygen properly handles anonymous
structs and enums that are typedef'd. file: [2b4aae9d25] check-in: [2fcf5a3052] user: js, branch: trunk, size: 7772 [annotate] [blame] [check-ins using] [diff]

ixemul is quite buggy and the native support we have for MorphOS is far
superior. file: [4c95dee7a0] check-in: [7ab180e9b6] user: js, branch: trunk, size: 8222 [annotate] [blame] [check-ins using] [diff]

MorphOS allows sharing a single socket base between tasks, so having a
per-task socket base is unnecessary. file: [fc5416a7db] check-in: [b45a563f2b] user: js, branch: trunk, size: 8278 [annotate] [blame] [check-ins using] [diff]

The internal hash is still 32 bit in most places, but this way, it is at
least not baked into the API and ABI and can be upgraded later, should
that ever be necessary. file: [750c66c5d1] check-in: [aeb403a1ed] user: js, branch: trunk, size: 8230 [annotate] [blame] [check-ins using] [diff]

Instead, provide getters and setters for the various address parts. file: [f684cec31b] check-in: [2ff4218405] user: js, branch: trunk, size: 8156 [annotate] [blame] [check-ins using] [diff]

This does not fully fix it yet, but at least the socket tests in the
test suite pass on 3DS now. file: [e3ee93c786] check-in: [ab89c47f42] user: js, branch: trunk, size: 5651 [annotate] [blame] [check-ins using] [diff]

This avoids the entire problem of keeping the buffer alive until the
send finished. file: [0d7cd9ede1] check-in: [0f3ae90eef] user: js, branch: trunk, size: 5610 [annotate] [blame] [check-ins using] [diff]

Instead, OFDNSResolver is used for everything now. file: [949bb8ad8d] check-in: [6e9ee122eb] user: js, branch: trunk, size: 5555 [annotate] [blame] [check-ins using] [diff]

This provides more convenience when the goal is to connect a socket and
provides functionality similar to getaddrinfo().

No support for resolving CNAMEs yet - this will be the next step. file: [2431bf7b16] check-in: [8555d64ee6] user: js, branch: trunk, size: 5249 [annotate] [blame] [check-ins using] [diff]

This pretends that net_getsockopt() exists, which it does not yet. As
soon as it lands in libogc, this will work again. file: [0a5ee032af] check-in: [bed2db0fd3] user: js, branch: trunk, size: 5125 [annotate] [blame] [check-ins using] [diff]

Instead of containing a struct sockaddr_storage - which does not exist
on all supported platforms - it now contains a union of all struct
sockaddr_* types.

Additionally, if struct sockaddr_in6 does not exist, it is defined so
that an IPv6 can be stored even if the system does not support IPv6. file: [d1393543d0] check-in: [8f810ecb7e] user: js, branch: trunk, size: 5102 [annotate] [blame] [check-ins using] [diff]

Not all OSes supported by ObjFW provide them. file: [232e64699e] check-in: [ab9810ef69] user: js, branch: trunk, size: 4355 [annotate] [blame] [check-ins using] [diff]

This replaces of_address_to_string_and_port(). file: [2892632dc8] check-in: [8aeee6680a] user: js, branch: trunk, size: 3792 [annotate] [blame] [check-ins using] [diff]

As we define the ABI, we can just replace BOOL with bool everywhere,
including in ObjFW itself. For the Apple platforms where BOOL and bool
are different, this is not a problem as BOOL and bool are passed and
returned the same way in the ABI.

This still defines BOOL to bool for compatibility, except on AmigaOS and
Wii, which both have its own BOOL type. file: [31842887c2] check-in: [fe2cbe0021] user: js, branch: trunk, size: 2233 [annotate] [blame] [check-ins using] [diff]

This reduces the ifdef hell and makes much more sense, as with ixemul,
no native calls are allowed. file: [1d3c37e76e] check-in: [c9621825fc] user: js, branch: trunk, size: 2323 [annotate] [blame] [check-ins using] [diff]

The previous approach of including <pthread.h> - which has the typedef -
turned out to be wrong, as that header is only a stub and including it
breaks things.

Since all the functions taking a socklen_t on other platforms take an
int on MorphOS, creating a typedef to int is the proper way. file: [40a01b9a4a] check-in: [d302df8b46] user: js, branch: trunk, size: 2037 [annotate] [blame] [check-ins using] [diff]

Strangely, MorphOS defines socklen_t in pthread.h. file: [61415b2d72] check-in: [69b3cfed8a] user: js, branch: trunk, size: 1978 [annotate] [blame] [check-ins using] [diff]

Forgot to add 2017, even though I already did quite some changes in
2017. file: [237f41c8f9] check-in: [44f45c2e35] user: js, branch: trunk, size: 1874 [annotate] [blame] [check-ins using] [diff]

This removes the port registry that existed for the Wii and instead
tries binding to random ports in a loop until it succeeds or errno is
not EADDRINUSE. This is done for the Wii and Nintendo 3DS now.

This also includes several other socket-related cleanups. file: [854e84e0a1] check-in: [12b8a7552e] user: js, branch: trunk, size: 1868 [annotate] [blame] [check-ins using] [diff]

While at it, also update the mail address. file: [0af78deb94] check-in: [cec0f072f8] user: js, branch: 0.8, size: 1836 [annotate] [blame] [check-ins using] [diff]

While at it, also update the mail address. file: [f7e5adc4a8] check-in: [2a27cf3000] user: js, branch: trunk, size: 2042 [annotate] [blame] [check-ins using] [diff]

This is necessary as the Wii does not allow picking a random free port,
and thus we need to track which ports are used. file: [6b8f6a13f5] check-in: [ed4549ddd3] user: js, branch: trunk, size: 2006 [annotate] [blame] [check-ins using] [diff]

This also adds -[isTCPNoDelayEnabled] and -[isKeepAliveEnabled],
together with OFGetOptionFailedException to indiciate failure to
retrieve the current value of the option. file: [f008ed8126] check-in: [12fbe85aab] user: js, branch: trunk, size: 1784 [annotate] [blame] [check-ins using] [diff]

of_getsockname() calls getsockname() after acquiring a mutex. This is
necessary as some implementations of getsockname() are not thread-safe. file: [3d1ffffaa0] check-in: [f18615e057] user: js, branch: trunk, size: 1632 [annotate] [blame] [check-ins using] [diff]

The old behaviour where the exception would access errno directly on
creation of the exception was very fragile. The two main problems with
it were that sometimes it would pick up an errno even though none had
been set and in other cases that when the exception was created errno
had already been overridden.

This also greatly increases errno handling on Win32, especially in
conjunction with sockets. It can still be improved further, though. file: [347ab62b21] check-in: [62e2de30b9] user: js, branch: trunk, size: 1496 [annotate] [blame] [check-ins using] [diff]

This is required as OFKernelEventObserver uses struct sockaddr_in as an
instance variable in case there is no pipe(). file: [bc099d612b] check-in: [4de1f2a67a] user: js, branch: trunk, size: 1370 [annotate] [blame] [check-ins using] [diff]

The new file socket.h includes all headers required for sockets on the
used platform, while the file socket_helpers.h defines the BSD API
functions to the platform specific functions if necessary.

This cleans up the classes dealing with sockets a lot and also reduces
code duplication. file: [2c4b5dfd5d] check-in: [81d47f4398] user: js, branch: trunk, size: 1113 [annotate] [blame] [check-ins using]

Endianess, size etc. may differ from system to system. While this might
be ok when writing files that are only read local again, this is
definitely not ok with sockets. So it's better to remove it from the
OFStream protocol. file: [e31807ae92] check-in: [70874cf3c5] user: js, branch: trunk, size: 2917 [annotate] [blame] [check-ins using] [diff]