ObjFW  History of src/OFUDPSocket.m of 1ba08eebc50c5214

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

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

There is a nasty bug in HP-UX: When using the proper socket APIs that
take a socklen_t (which is 64 bit in LP64 mode), it still calls into
methods that expect an int - so there is a type mismatch between the
headers and the compiled system libraries. This leads to all socklen_t
being interpreted as 0 (due to big endian). Defining
_HPUX_ALT_XOPEN_SOCKET_API makes it use the correct symbols in the
system libraries instead. file: [26835c8d41] check-in: [1833194867] user: js, branch: trunk, size: 5054 [annotate] [blame] [check-ins using] [diff]

This is intended for SCTP and SPX. file: [2b709ddc80] check-in: [e5b2ee56ff] user: js, branch: trunk, size: 4966 [annotate] [blame] [check-ins using] [diff]

This makes it easier to reuse code for other datagram sockets, namely
IPX. file: [62f6a25c2d] check-in: [b02c4b05bb] user: js, branch: trunk, size: 4891 [annotate] [blame] [check-ins using] [diff]

This temporarily doesn't use the search domains anymore and makes CNAMEs
slightly more inefficient. The next commits will fix both. file: [de0f80ba6e] check-in: [ee592cf603] user: js, branch: trunk, size: 11923 [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: [08c14deedf] check-in: [ab89c47f42] user: js, branch: trunk, size: 11896 [annotate] [blame] [check-ins using] [diff]

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

Instead, OFDNSResolver is used for everything now. file: [073268c5ca] check-in: [6e9ee122eb] user: js, branch: trunk, size: 12389 [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: [06139f0b65] check-in: [bed2db0fd3] user: js, branch: trunk, size: 14277 [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: [1e4bb48119] check-in: [8f810ecb7e] user: js, branch: trunk, size: 14217 [annotate] [blame] [check-ins using] [diff]

This has always been wrong. file: [6754e57a6e] check-in: [e36455441d] user: js, branch: trunk, size: 12961 [annotate] [blame] [check-ins using] [diff]

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

Instead, explicitly declare them, as OF_ASSUME_NONNULL_{BEGIN,END} does
not apply to implicit return types. This means that after this commit,
all init methods have a nonnull return type, as they should have. file: [8a3b636da1] check-in: [2f4e0df8be] user: js, branch: trunk, size: 15924 [annotate] [blame] [check-ins using] [diff]

This allows retrieving the number of bytes already written before the
write failed, allowing to retry without writing data that has already
been written. file: [c8e3c83553] check-in: [2ae01218ef] user: js, branch: trunk, size: 14860 [annotate] [blame] [check-ins using] [diff]

Previously, socket_helpers.h would just define close() to whatever is
needed to close a socket. However, this makes it possible to
accidentally call a socket-specific close on something that is not a
socket. Having to explicitly call closesocket() (and defining it to
close() on platforms where it's the same) prevents this. file: [83754f70d8] check-in: [b4cd87cd81] user: js, branch: trunk, size: 14549 [annotate] [blame] [check-ins using] [diff]

This matches Apple's style. file: [06ff6d8241] check-in: [6b77a5dd8b] user: js, branch: trunk, size: 14519 [annotate] [blame] [check-ins using] [diff]

Casts are now written like types in variable declarations. file: [f67659d08d] check-in: [4af49a13c3] user: js, branch: trunk, size: 14519 [annotate] [blame] [check-ins using] [diff]

Forgot to add 2017, even though I already did quite some changes in
2017. file: [7e2210cd91] check-in: [44f45c2e35] user: js, branch: trunk, size: 14478 [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: [ad50b942db] check-in: [12b8a7552e] user: js, branch: trunk, size: 14423 [annotate] [blame] [check-ins using] [diff]

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

While at it, also update the mail address. file: [0280b236b8] check-in: [2a27cf3000] user: js, branch: trunk, size: 13800 [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: [b7f838da94] check-in: [ed4549ddd3] user: js, branch: trunk, size: 13789 [annotate] [blame] [check-ins using] [diff]

This fixes the tests on Wii. file: [182b0b3dfd] check-in: [4cecf82254] user: js, branch: trunk, size: 13205 [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: [455c967f24] check-in: [f18615e057] user: js, branch: trunk, size: 12874 [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: [3c38516b1b] check-in: [62e2de30b9] user: js, branch: trunk, size: 12842 [annotate] [blame] [check-ins using] [diff]

This means that OFObject.h imports macros.h now, making it unnecessary
to manually import macros.h in almost every file. And while at it, also
import autorelease.h in OFObject.h, so that this doesn't need to be
manually imported in almost every file as well. file: [2b1705fa68] check-in: [13ee56edf3] user: js, branch: trunk, size: 12231 [annotate] [blame] [check-ins using] [diff]

LLP64 was mostly fast enumeration using an unsigned long for the state,
which can't store a pointer or a size_t on LLP64. This is now solved by
either throwing an OFOutOfRangeException if the value of the size_t is
bigger than ULONG_MAX or storing the pointer in the extra field (copied
using memcpy, as it's an array of unsigned long, which again would be
too small to store a pointer).

Win64 was mostly Microsoft not being able to decide whether a length is
a size_t, a DWORD, an int or an unsigned int (thus the different types
in places that seem to be almost the same). But since that would not be
confusing enough, a file descriptor is an int if it's for a file, but a
long long if it is for a socket. But of course, for ReadFile and friends
it's a DWORD instead of an int then. file: [473e259227] check-in: [4e59d2692f] user: js, branch: trunk, size: 12274 [annotate] [blame] [check-ins using] [diff]

No functional change intended. file: [9fe4ea7a6f] check-in: [9da9b2b64c] user: js, branch: trunk, size: 11390 [annotate] [blame] [check-ins using] [diff]

It caused a warning about an implicit cast on OS X/PPC64. file: [aca7fe3ad4] check-in: [3fa37632f9] user: js, branch: trunk, size: 11385 [annotate] [blame] [check-ins using] [diff]

It is now called +[getHost:andPort:forAddress:]. This makes it much
clearer from the selector alone what it actually does. file: [584c86fff6] check-in: [fb590316ba] user: js, branch: trunk, size: 6996 [annotate] [blame] [check-ins using] [diff]

Async operations will follow later. file: [72deb2c0c0] check-in: [5025cba435] user: js, branch: trunk, size: 6856 [annotate] [blame] [check-ins using]