ObjFW  History of src/OFStdIOStream.m of f2e96ebd9a89f730

Use "\r" instead of the ANSI escape code if the column is 0.

This fixes the output of the tests on 3DS. file: [4854e57444] check-in: [1c2fe75e6e] user: js, branch: trunk, size: 13633 [annotate] [blame] [check-ins using] [diff]

A better approach would probably be to have a separate header which
makes some classes available to pure C. file: [f8e458060e] check-in: [14c70763b0] user: js, branch: trunk, size: 11322 [annotate] [blame] [check-ins using] [diff]

All necessary changes to adjust for the changes made in trunk are
included in the merge commit. file: [2787693b17] check-in: [3c88df0ce4] user: js, branch: amiga-library, size: 10419 [annotate] [blame] [check-ins using] [diff]

When having Xcode attached to see the log, isatty() returns true, but
does not handle any control sequences. file: [10bbd8e570] check-in: [a98ad80cd3] user: js, branch: trunk, size: 10185 [annotate] [blame] [check-ins using] [diff]

This is done by using functions that return a reference to the variable
and defining of_std{in,out,err} to a call into these functions. file: [acfa5ef390] check-in: [9697334084] user: js, branch: amiga-library, size: 10804 [annotate] [blame] [check-ins using] [diff]

This is required for the Amiga library. file: [3648853d0b] check-in: [343501981b] user: js, branch: amiga-library, size: 10516 [annotate] [blame] [check-ins using] [diff]

This means refusing to close twice, calling -[close] from -[dealloc] and
not calling -[cancelAsyncRequests].

Calling -[cancelAsyncRequests] in -[close] is too dangerous, as -[close]
gets called by -[dealloc]: If the queue is the last reference to the
object, at the point where -[cancelAsyncRequests] removes it from the
queue, the object will start to deallocate and call into
-[cancelAsyncRequests] again, which is still in the middle of removing
it and now finds itself with an inconsistent state. file: [730550a384] check-in: [3b43d51006] user: js, branch: trunk, size: 7419 [annotate] [blame] [check-ins using] [diff]

Previously, this just assumed the existance of file descriptors 0, 1 and
2 - however, in GUI applications on Windows, these were missing when not
started from a console. file: [78d2467bd8] check-in: [0f8a72d43c] user: js, branch: trunk, size: 7919 [annotate] [blame] [check-ins using] [diff]

libauto loads the libraries too late. file: [c5d7e54168] check-in: [d71abc5366] user: js, branch: trunk, size: 7738 [annotate] [blame] [check-ins using] [diff]

This has been fixed upstream in amiga-gcc. file: [5e211fc7f6] check-in: [b31bd48f90] user: js, branch: trunk, size: 6965 [annotate] [blame] [check-ins using] [diff]

The reason for this is that, in theory, this also supports versions
older than 3. file: [3ca31d2e74] check-in: [5831626f68] user: js, branch: trunk, size: 7027 [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: [7f1468fef3] check-in: [fe2cbe0021] user: js, branch: trunk, size: 6965 [annotate] [blame] [check-ins using] [diff]

This is the first step to have different versions of those for different
operating systems, rather than #ifdefs everywhere.

This also has the nice side-effect of not having one implementation in
OFString and another one in OFString_UTF8 anymore. The one in OFString
was a generic version, while the one in OFString_UTF8 was one optimized
for UTF-8 strings. Now only the version optimized for UTF-8 strings
exists, as this is by far the most common string implementation used,
and the overhead of converting from something else to UTF-8 is not more
than converting to UTF-32. file: [66757147ba] check-in: [7a488c3062] user: js, branch: trunk, size: 7002 [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: [3619228f8c] check-in: [2f4e0df8be] user: js, branch: trunk, size: 6910 [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: [cf1f27b3cb] check-in: [2ae01218ef] user: js, branch: trunk, size: 7025 [annotate] [blame] [check-ins using] [diff]

This removes several initializers that omitted the errNo. Removing those
forces to think about whether there is a meaningful errNo to set instead
of just omitting it. file: [ed0f11c59f] check-in: [4f36894ce7] user: js, branch: trunk, size: 6671 [annotate] [blame] [check-ins using] [diff]

Instead, let reads return 0 and let writes append after the end. file: [fac79ea11f] check-in: [f9cd4f9cab] user: js, branch: trunk, size: 6626 [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: [87d19ba681] check-in: [c9621825fc] user: js, branch: trunk, size: 6784 [annotate] [blame] [check-ins using] [diff]

It turns out that including unistd.h does work when defining BOOL to
something else. file: [26c295bea1] check-in: [cb450cb1bd] user: js, branch: trunk, size: 7118 [annotate] [blame] [check-ins using] [diff]

This is necessary for using -noixemul. file: [b0cc705195] check-in: [614c6678e8] user: js, branch: trunk, size: 5341 [annotate] [blame] [check-ins using] [diff]

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

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

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

Right now, this is only useful for OFTCPSocket, as this is the only
class so far not establishing the stream in the init method. However,
this adds the general infrastructure to allow reuse to all subclasses of
OFStream. file: [3491f0262a] check-in: [1de551cb5f] user: js, branch: trunk, size: 4304 [annotate] [blame] [check-ins using] [diff]

This is required for static linking, as otherwise
+[OFStdIOStream_Win32Console load] will not be called. file: [53b11ea420] check-in: [e53a4fd61c] user: js, branch: trunk, size: 4287 [annotate] [blame] [check-ins using] [diff]

The previous way was to set the codepage to UTF-8, however, this does
not work properly on some versions of Windows.

Instead, this catches reads / writes on of_std* on the lowlevel,
interprets it as UTF-8, converts it to / from UTF-16 and then uses
ReadConsoleW() / WriteConsoleW().

Surrogates being cut in the middle is not properly handled yet, this
will be implemented in a follow up commit. file: [e2adf5fa8a] check-in: [3e1b6bccbc] user: js, branch: trunk, size: 4066 [annotate] [blame] [check-ins using] [diff]

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

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

Now that we can require GCC >= 4.6, we no longer need to fall back to
using a category on old compilers. file: [b89433fb35] check-in: [2aca549d60] user: js, branch: trunk, size: 4088 [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: [a3654b4d40] check-in: [62e2de30b9] user: js, branch: trunk, size: 4023 [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: [3cbe48a8b1] check-in: [13ee56edf3] user: js, branch: trunk, size: 3783 [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: [84af7e353c] check-in: [4e59d2692f] user: js, branch: trunk, size: 3662 [annotate] [blame] [check-ins using] [diff]

This is in preparation for adding UDP sockets, as UDP sockets and TCP
sockets have no common superclass, as one is stream-oriented while the
other is packet-oriented.

Read and write exceptions are for any object now, as they are useful for
a lot more than just for streams, while the others (bind, listen, etc.)
are for any socket now (the type is id in this case, though, as there is
no common superclass). file: [5073a1b1d8] check-in: [8d2a5052fd] user: js, branch: trunk, size: 3060 [annotate] [blame] [check-ins using] [diff]

This mostly removes the argument for the class in which the exception
occurred. As backtraces were recently added for all platforms, the
passed class does not give any extra information on where the exception
occurred anymore.

This also removes a few other arguments which were not too helpful. In
the past, the idea was to pass as many arguments as possible so that it
is easier to find the origin of the exception. However, as backtraces
are a much better way to find the origin, those are not useful anymore
and just make the exception more cumbersome to use. The rule is now to
only pass arguments that might help in recovering from the exception or
provide information that is otherwise not easily accessible. file: [0fb850ba21] check-in: [3d16a30f41] user: js, branch: trunk, size: 3048 [annotate] [blame] [check-ins using] [diff]

This is used for of_std{in,out,err} now instead of OFFile. file: [f5689e7f1a] check-in: [085bfc12a0] user: js, branch: trunk, size: 3105 [annotate] [blame] [check-ins using]