ObjFW  History of src/OFApplication.m of f2e96ebd9a89f730

While we don't need argc/argv when using the wide variants on Windows,
an application might need to retrieve them via
-[getArgumentCount:andArgumentValues:] for passing them off to another
library. file: [d30209c2bc] check-in: [2ca00fefff] user: js, branch: trunk, size: 16197 [annotate] [blame] [check-ins using] [diff]

This is only migrating some places, others will be migrated as they are
touched. file: [84a11f1539] check-in: [a0fd103a0b] user: js, branch: trunk, size: 15740 [annotate] [blame] [check-ins using] [diff]

It is unclear whether the same API can be applied to other sandboxing
frameworks. file: [7562fe05db] check-in: [510628432f] user: js, branch: 1.0, size: 15915 [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: [9bf3f4dbf0] check-in: [b45a563f2b] user: js, branch: trunk, size: 15897 [annotate] [blame] [check-ins using] [diff]

While the active sandbox can be changed, a different sandbox must not be
activated. The reason for this is that allowing to activate a different
sandbox makes it impossible to track which paths have already been
unveiled. file: [bd6d38d7b4] check-in: [e0c2e70f7b] user: js, branch: trunk, size: 14794 [annotate] [blame] [check-ins using] [diff]

When changing and activating a sandbox again, do not unveil paths that
have already been unveiled by the previous activation. file: [517256bc85] check-in: [8af3eedb02] user: js, branch: trunk, size: 14649 [annotate] [blame] [check-ins using] [diff]

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

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

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

There is no reliable way to check if C++ already defined those, as Clang
now defines char{16,32}_t even when it is not in C++ 11 mode. So we
cannot check if we are not in C++ 11 and then define them and otherwise
rely on the C++ 11 types. file: [240e542914] check-in: [505137f25f] user: js, branch: trunk, size: 12689 [annotate] [blame] [check-ins using] [diff]

This is done at compile time now rather than at runtime. file: [9f443dae6c] check-in: [54b38752bb] user: js, branch: trunk, size: 12663 [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: [c080ac0fe6] check-in: [2f4e0df8be] user: js, branch: trunk, size: 13108 [annotate] [blame] [check-ins using] [diff]

This way, the behavior is consistent, independent of whether sigaction()
is available or not. file: [a37acbc186] check-in: [4557cfb743] user: js, branch: trunk, size: 13040 [annotate] [blame] [check-ins using] [diff]

This removes "b" for binary and always uses binary, as there is no good
reason to not use binary. file: [c9d4f7d315] check-in: [23e57c5040] user: js, branch: trunk, size: 13650 [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: [970d1b40a3] check-in: [c9621825fc] user: js, branch: trunk, size: 13651 [annotate] [blame] [check-ins using] [diff]

Double linked lists on MorphOS/AmigaOS have the last node point back to
the list head, so checking the successor for NULL is not correct, as
this will then include the list head which has no payload. file: [cd6046dfba] check-in: [22b2e6ea8b] user: js, branch: trunk, size: 13726 [annotate] [blame] [check-ins using] [diff]

This is required for -noixemul on MorphOS. file: [fd2efe060c] check-in: [ddf0fae30d] user: js, branch: trunk, size: 11836 [annotate] [blame] [check-ins using] [diff]

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

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

Only OpenBSD's pledge() is supported so far. file: [4ef7cfc06b] check-in: [40cf53506a] user: js, branch: trunk, size: 11680 [annotate] [blame] [check-ins using] [diff]

DJGPP has sigaction, but not SA_RESTART. file: [735d2c61f2] check-in: [cb0a90f26d] user: js, branch: trunk, size: 11039 [annotate] [blame] [check-ins using] [diff]

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

This singleton gives access to all things locale, including the ability
to get localized strings.

This also adds the OF_LOCALIZED() macro. Its first argument is an ID for
the string to be localized and its second argument is the fallback
string to be used if it cannot retrieve the localized string. Following
that are variable name / value pairs to be replaced in the localized
string.

Getting translated strings is not implemented yet: Instead, it always
uses the fallback string.

This also switches ofhttp to localized strings. file: [514600c380] check-in: [06bcb21fc7] user: js, branch: trunk, size: 10952 [annotate] [blame] [check-ins using] [diff]

Yes, there is really nothing to query this.
Yes, locales in C and POSIX are broken way beyond repair. file: [4b696ca492] check-in: [d6de0a1b71] user: js, branch: trunk, size: 11056 [annotate] [blame] [check-ins using] [diff]

This is achieved by replacing the locale's decimal point with "." after
formatting and replacing "." with the locale's decimal point before
parsing.

To still use the decimal point from the locale for formatting, the new
flag "," is introduced to formats. This is useful for just printing a
string to the user that is not saved to a file or sent via a network.

While this is an ugly hack, there is no better way to do this other than
implementing the functionality of printf and strtod myself, as POSIX
does not specify versions of these that take a locale as an argument.
While this is a lot of work and error-prone, I will most likely end up
doing this eventually.

This commit also enables the locale in OFApplication to notice when
things break. As a nice side effect, ofhttp now uses the locale's
decimal point in its user interface. file: [669a08ac5c] check-in: [6dff0f5922] user: js, branch: trunk, size: 10856 [annotate] [blame] [check-ins using] [diff]

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

While at it, also update the mail address. file: [932d52d201] check-in: [2a27cf3000] user: js, branch: trunk, size: 10740 [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: [22742689c9] check-in: [2aca549d60] user: js, branch: trunk, size: 10736 [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: [faff25c759] check-in: [48980f2297] user: js, branch: trunk, size: 10755 [annotate] [blame] [check-ins using] [diff]

This allows setting SA_RESTART. file: [a553adbdf6] check-in: [8bc06182a4] user: js, branch: trunk, size: 10835 [annotate] [blame] [check-ins using] [diff]

This replaces +[OFString nativeOSEncoding]. file: [3fb6075d12] check-in: [ec66e49dca] user: js, branch: trunk, size: 10219 [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: [560f80331f] check-in: [13ee56edf3] user: js, branch: trunk, size: 10161 [annotate] [blame] [check-ins using] [diff]

This replaces OF_STRING_ENCODING_NATIVE, as a define was not flexible
enough (determining the native OS encoding at startup was not possible). file: [ce308c71b7] check-in: [e54c8c0368] user: js, branch: trunk, size: 9947 [annotate] [blame] [check-ins using] [diff]

A file documenting the coding style will be written soon. This will
hopefully prevent conflicts in the future, such as whether static
functions are written in camelCase or_with_underscores, like was the
case here. file: [c9284fd43c] check-in: [fa6496efc7] user: js, branch: trunk, size: 9890 [annotate] [blame] [check-ins using] [diff]

Includes:
* Check environ against NULL before using it, since Android sets it to
NULL.
* Cast st_size of struct stat to off_t. Android uses long long for
st_size as its off_t is only 32 bit, but st_size should be off_t
according to POSIX.
* Android's strtod() does not accept 0x, as specified by C99. Thus,
the test is disabled if __ANDROID__ is defined. file: [91fefa5006] check-in: [c5e7dd679b] user: js, branch: trunk, size: 9892 [annotate] [blame] [check-ins using] [diff]

It's safe to always import OFThread.h now as OFThread even offers
limited functionality when threads are unavailable. file: [15015abb7a] check-in: [d7365be372] user: js, branch: trunk, size: 9369 [annotate] [blame] [check-ins using] [diff]

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

It seems subclassing OFApplication instead of OFObject for the
application delegate is a common mistake. file: [ca5bc2d367] check-in: [fb79b7f369] user: js, branch: trunk, size: 9367 [annotate] [blame] [check-ins using] [diff]

It still stores the normal argc/argv for compatibility, however, the
argument array is created from the Unicode arguments instead.

This also makes -[OFApplication setArgumentCount:andArgumentValues:]
private. file: [09b9e89928] check-in: [fb1f29bd30] user: js, branch: trunk, size: 8473 [annotate] [blame] [check-ins using] [diff]

Before, when the application would exit after
-[applicationDidFinishLaunching], this was unnecessary. But now, without
this change, temporary objects might be kept alive while the run loop is
active. file: [c50e9dd27b] check-in: [bba061e4b7] user: js, branch: trunk, size: 6966 [annotate] [blame] [check-ins using] [diff]

This is in preparation for adding an event loop, so that applications
which do not require a run loop will already adopt to calling
[OFApplication terminate] at the end of
-[applicationDidFinishLaunching]. file: [a06fbcf0e7] check-in: [24622e8913] user: js, branch: trunk, size: 6755 [annotate] [blame] [check-ins using] [diff]

This greatly improves performance, as it gets rid of the overhead of
OFAutoreleasePool. file: [77883af432] check-in: [1255f3a11a] user: js, branch: trunk, size: 6300 [annotate] [blame] [check-ins using] [diff]

There is no need to reference count them, they will always be retained
by the application for the lifetime of the application. file: [64dece92ce] check-in: [9ec315d070] user: js, branch: trunk, size: 5177 [annotate] [blame] [check-ins using] [diff]

This is safe as an "exception loop" can't happen, since if allocating
an exception fails, it throws an OFAllocFailedException which is
preallocated and can always be thrown.

So, the worst case would be that an autorelease of an exception fails,
triggering an OFOutOfMemoryException for which there is no memory,
resulting in an OFAllocFailedException to be thrown. file: [e705f7e9ee] check-in: [e1e7ffa903] user: js, branch: trunk, size: 5271 [annotate] [blame] [check-ins using] [diff]

This makes it possible to pass argc and argv to third party libs
requiring those. file: [52b96796ce] check-in: [f282aaa672] user: js, branch: trunk, size: 3672 [annotate] [blame] [check-ins using] [diff]

We assume now that every object understands retain, release, etc. so
that we can weaken the type from OFObject* to id. This makes it
possible to use different root object classes. file: [4cc1491a54] check-in: [e2b06423e0] user: js, branch: trunk, size: 3553 [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: [cbba891989] check-in: [37db8f0fb3] user: js, branch: trunk, size: 2392 [annotate] [blame] [check-ins using] [diff]