ObjFW  History of src/OFTCPSocket.h of 8939cbdb52a79823

Unfortunately, I haven't found a way to reliably check if a connection
is using MPTCP on macOS/iOS yet, so -[usesMPTCP] always returns false
for now. In my tests with SIOCGCONNINFO, the flags were always 0. file: [a5332f3211] check-in: [4e763857b8] user: js, branch: mptcp, size: 9082 [annotate] [blame] [check-ins using] [diff]

In order to not accidentally have C strings with \0, an
OFInvalidEncodingException is thrown when trying to get a C string for a
string that contains \0.

In order to get a C string with \0 anyway, a new method
-[insecureCStringWithEncoding:] is added. file: [c7a38bf638] check-in: [6fbc3b18c7] user: js, branch: trunk, size: 8784 [annotate] [blame] [check-ins using] [diff]

This should make it easier to add TLS support using various
implementations. file: [4ad589837f] check-in: [34cb121dc5] user: js, branch: trunk, size: 6183 [annotate] [blame] [check-ins using] [diff]

__COUNTER__ does not exist in GCC 4.2, and Apple GCC 4.2 is still the
newest compiler available for macOS 10.5. file: [f9723931da] check-in: [ef614a225d] user: js, branch: trunk, size: 6424 [annotate] [blame] [check-ins using] [diff]

Arguments that can just be captured by the block don't need to be
repeated as an argument. A lot of arguments passed to a delegate are
therefore not needed for a block. file: [8293d08642] check-in: [d7ddb3dbc7] user: js, branch: trunk, size: 6418 [annotate] [blame] [check-ins using] [diff]

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

This is a new superclass for OFTCPSocket to allow sharing code with
other IP stream socket types, namely SCTP. file: [3e0d4559a3] check-in: [3dfe642dd3] user: js, branch: trunk, size: 3336 [annotate] [blame] [check-ins using] [diff]

This allows ABI stability with the fragile ABI. file: [52ed0f333c] check-in: [496695d778] user: js, branch: trunk, size: 9312 [annotate] [blame] [check-ins using] [diff]

Otherwise, there would be two methods for every operation: One for
success and one for failure. It also makes it easy to forget about
handling failure, so it's better to always pass an optional exception. file: [e932bc199a] check-in: [064dbe5127] user: js, branch: trunk, size: 9360 [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: [0973252c95] check-in: [8f810ecb7e] user: js, branch: trunk, size: 7314 [annotate] [blame] [check-ins using] [diff]

This improves Swift compatibility. file: [4bf4a25fa2] check-in: [a2e849c681] user: js, branch: trunk, size: 7292 [annotate] [blame] [check-ins using] [diff]

This improves Swift compatibility. file: [710f0fdfb9] check-in: [f3ccf14461] user: js, branch: trunk, size: 7135 [annotate] [blame] [check-ins using] [diff]

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

This changes retaining behavior, meaning properties are not returned
retained and autoreleased anymore, so a property returned from a getter
now needs to be manually retained and autoreleased before calling the
setter. However, this is rarely the case and not using atomic improves
performance. file: [7e829b3ab1] check-in: [0a73af49f0] user: js, branch: trunk, size: 7520 [annotate] [blame] [check-ins using] [diff]

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

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

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

This increases the required GCC version from 4.0 to 4.6 (exception:
Apple GCC, which already supports this with >= 4.0 starting with OS X
10.5). Since even GCC 4.6 is really old by now, there is no point in
still supporting something even older and making the code ugly because
of that. While some hardware and OS support was dropped from GCC 4.6
compared to GCC 4.0, there is nothing in there that would be an
interesting target with the exception of BeOS maybe - but a port to BeOS
can also be achieved using the Haiku support. The other dropped OSes are
mostly old versions of OSes while newer ones are still being supported
(and those newer versions of those OSes still support the same
hardware). file: [694cce421a] check-in: [48980f2297] user: js, branch: trunk, size: 7235 [annotate] [blame] [check-ins using] [diff]

Now that Clang switched to use _Nullable and _Nonnull instead of
__nullable / __nonnull, there is no longer a conflict with glibc, which
means we can just define _Nullable / _Nonnull to nothing if they are not
understood by the compiler (which did not work with __nullable /
__nonnull due to this conflict).

This also defines _Null_unspecified to nothing if unavailable. file: [68e32c48a9] check-in: [d526d938d7] user: js, branch: trunk, size: 7978 [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: [55b987fd74] check-in: [ed4549ddd3] user: js, branch: trunk, size: 7985 [annotate] [blame] [check-ins using] [diff]

This is necessary to make GCC (>= 4.6 and Apple GCC) happy as they don't
like the list of property attributes ending in a comma. file: [6382beb2ce] check-in: [80e1a9655b] user: js, branch: trunk, size: 7901 [annotate] [blame] [check-ins using] [diff]

Using __nonnull directly doesn't work on systems using glibc, as glibc
defines __nonnull as a parameterized define. While this does not fix the
problem of Clang introducing __nonnull even though it conflicts with
glibc, this at least means it's possible again to compile things with
versions of Clang that don't support __nonnull on systems with glibc. file: [5654a78b4e] check-in: [b7097a67b6] user: js, branch: trunk, size: 7890 [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: [0e29d56685] check-in: [12fbe85aab] user: js, branch: trunk, size: 7751 [annotate] [blame] [check-ins using] [diff]

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

This is in preparation for UDP sockets, so that they can share the
resolving functionality with TCP sockets. file: [4d85340515] check-in: [49b1989b63] user: js, branch: trunk, size: 7014 [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: [4b861449bf] check-in: [81d47f4398] user: js, branch: trunk, size: 7020 [annotate] [blame] [check-ins using] [diff]

Not functional yet due to bugs in the SDK.
Bugs found so far:

* Binding to port 0 fails instead of choosing a free port.
* gethostbyname() does not work for IPs.
* getsockname() is missing.
* struct sockaddr_storage is missing.

I have not decided yet whether I fix those bugs in the SDK (I already
implemented getsockname() and added struct sockaddr_stroage and it seems
to work) or if I work around them in ObjFW. This will mainly depend on
how cooperative the developers of the Wii SDK are. file: [2b3fb6a25a] check-in: [ffb91daffe] user: js, branch: trunk, size: 7341 [annotate] [blame] [check-ins using] [diff]

This also removes "bool *stop" from some blocks where it does not make
sense. file: [efe8886d8a] check-in: [e8473b9db8] user: js, branch: trunk, size: 7267 [annotate] [blame] [check-ins using] [diff]

The only places where BOOL is left are those where they are required by
the ABI. file: [b4d76328d1] check-in: [c5ef582958] user: js, branch: trunk, size: 6583 [annotate] [blame] [check-ins using] [diff]

This removes of_http_client_tls_socket_class in favor of having one
symbol for all places where TLS would be desired. file: [7bc1670242] check-in: [1abe5a111f] user: js, branch: trunk, size: 6512 [annotate] [blame] [check-ins using] [diff]

This makes it possible to handle exceptions that occurred during async
I/O in the specified handler. file: [facc710023] check-in: [6343e04dcd] user: js, branch: trunk, size: 6408 [annotate] [blame] [check-ins using] [diff]

Async connecting is still missing. file: [804b5d55d9] check-in: [4d9a641d87] user: js, branch: trunk, size: 5800 [annotate] [blame] [check-ins using] [diff]

For example, this is useful if you want to use
+[OFString stringWithContentsOfURL:] with a proxy or if you just want
to use anything that uses an OFTCPSocket with a proxy (even 3rd party
libraries). file: [fa3ba9a6c5] check-in: [a3b3333128] user: js, branch: trunk, size: 4212 [annotate] [blame] [check-ins using] [diff]

This is to reflect that it is a connection-based, sequenced, reliable,
two-way byte stream. file: [b07d89367b] check-in: [28ea4a8a67] user: js, branch: trunk, size: 1802 [annotate] [blame] [check-ins using] [diff]

ivars are now named like this: thisIsAnIVar. If an accessor exists for
the ivar, the ivar is named like the accessor. This is required for
@property to work. file: [19e54da62b] check-in: [37db8f0fb3] user: js, branch: trunk, size: 1790 [annotate] [blame] [check-ins using] [diff]