ObjFW  History of src/OFDate.m of 452bdb4a5f026d53

Also fixes Doxygen not creating documentation for `static OF_INLINE`
functions. file: [ddae0ab707] check-in: [5164cbc576] user: js, branch: 1.1, size: 17931 [annotate] [blame] [check-ins using] [diff]

Also fixes Doxygen not creating documentation for `static OF_INLINE`
functions. file: [9e5a69c33e] check-in: [0c3c3b6efa] user: js, branch: trunk, size: 17930 [annotate] [blame] [check-ins using] [diff]

These places are not super performance critical and the code becomes a
lot more readable.

Actually using @synchronized in ObjFW itself is also a good motivation
to optimize it further. file: [2047693bf0] check-in: [ff3a60cc12] user: js, branch: trunk, size: 17918 [annotate] [blame] [check-ins using] [diff]

While the idea sounds nice that the tag name is the class, this means the
serialization includes whether something is mutable or immutable. This means
doing as much as making something immutable changes the serialization, which
can then cause issues after being deserialized. file: [8b8a822187] check-in: [6ce0093f8d] user: js, branch: trunk, size: 18818 [annotate] [blame] [check-ins using] [diff]

In our tests, we call objc_exit() using atexit(), so the runtime is
already gone by the time any destructors get called. file: [3c5313a6f3] check-in: [9cedc18bb3] user: js, branch: trunk, size: 20155 [annotate] [blame] [check-ins using] [diff]

The new names should be more accurate. file: [2f04475c25] check-in: [627511b032] user: js, branch: trunk, size: 20085 [annotate] [blame] [check-ins using] [diff]

While it makes sense to wrap malloc and calloc to replace the error
checking with exceptions, it does not make sense to wrap free. file: [cf1b1da641] check-in: [3d8286feee] user: js, branch: trunk, size: 20087 [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: [f5afc23b94] check-in: [aeb403a1ed] user: js, branch: trunk, size: 20168 [annotate] [blame] [check-ins using] [diff]

This solves the old signed vs. unsigned problem and allows for more
bases than just 8, 10 and 16, as well as auto-detection of the base (if
base is 0). file: [c70c88b347] check-in: [b6ee372b98] user: js, branch: trunk, size: 16109 [annotate] [blame] [check-ins using] [diff]

This makes old versions of Clang happy which don't know about
-Wobjc-designated-initializers. file: [8d4b356936] check-in: [94c7e3e8f3] user: js, branch: tagged-pointers, size: 19796 [annotate] [blame] [check-ins using] [diff]

This means all other initializers use this initializer, which means all
other initializers can be reused for tagged pointers. file: [7bc28080b9] check-in: [33a133cda0] user: js, branch: tagged-pointers, size: 18234 [annotate] [blame] [check-ins using] [diff]

This replaces all ivar accesses with calls to -[timeIntervalSince1970]
(except for initializers and -[timeIntervalSince1970] itself, which is
the only method accessing an ivar and will be overridden for tagged
pointers), returns a placeholder for +[alloc] and makes +[distantFuture]
and +[distantPast] singletons. file: [81bcdf5816] check-in: [95830bf78b] user: js, branch: tagged-pointers, size: 18751 [annotate] [blame] [check-ins using] [diff]

floor() does not have the desired effect here when the number is
negative. file: [a3e5369167] check-in: [3c20dd5f95] user: js, branch: trunk, size: 15598 [annotate] [blame] [check-ins using] [diff]

Since OFComparing no longer conforms to OFObject, OFObject needed to be
added at a few places. file: [da50e80816] check-in: [11f24addd7] user: js, branch: trunk, size: 14134 [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: [f5a435fe1e] check-in: [2f4e0df8be] user: js, branch: trunk, size: 14087 [annotate] [blame] [check-ins using] [diff]

Some time zones have an offset that is not just full hours. file: [618922b043] check-in: [da9b9d7e3b] user: js, branch: trunk, size: 13999 [annotate] [blame] [check-ins using] [diff]

This way, they are no longer special cases. file: [388aebd154] check-in: [3ba1a97679] user: js, branch: trunk, size: 13948 [annotate] [blame] [check-ins using] [diff]

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

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

This fixes including a non-module header in a module. file: [99b4552175] check-in: [9c108208ef] user: js, branch: trunk, size: 14429 [annotate] [blame] [check-ins using] [diff]

This adds %z for parsing time zones. Additionally, this also adds %a and
%b. file: [6c84c9ff36] check-in: [e4439b7ef8] user: js, branch: trunk, size: 14281 [annotate] [blame] [check-ins using] [diff]

It's better to make sure we never round up, as it's better to be a
little bit behind in time than ahead.

This has the nice side effect of fixing the test on the 3DS, as lrint()
is behaving differently there. file: [d7cb394675] check-in: [f1765412c7] user: js, branch: trunk, size: 13690 [annotate] [blame] [check-ins using] [diff]

(U)INT_FAST*_MAX is broken on Android, which makes (u)int_fast*_t
dangerous to use while it does not provide any real speed benefit. file: [69ce62e752] check-in: [d3158d091f] user: js, branch: trunk, size: 13699 [annotate] [blame] [check-ins using] [diff]

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

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

strftime creates strings on Windows that are not valid UTF-8. file: [a9bad35577] check-in: [f585645c21] user: js, branch: 0.8, size: 13684 [annotate] [blame] [check-ins using] [diff]

strftime creates strings on Windows that are not valid UTF-8. file: [edf5bb546e] check-in: [b8fc41bf18] user: js, branch: trunk, size: 13700 [annotate] [blame] [check-ins using] [diff]

Using -std=c* causes all kinds of problems with different OSes (some
need a _GNU_SOURCE define when using -std=c*, others simply don't
declare some functions at all due to __STRICT_ANSI__ being defined by
-std=c*). file: [e749446c20] check-in: [989363c9fd] user: js, branch: trunk, size: 12709 [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: [58d1e162c8] check-in: [13ee56edf3] user: js, branch: trunk, size: 12699 [annotate] [blame] [check-ins using] [diff]

Instead, just define _GNU_SOURCE when __GLIBC__ is defined. After all,
that's the only libc that doesn't work properly without any defines. file: [99de8e6f4d] check-in: [17be12b6e5] user: js, branch: trunk, size: 12736 [annotate] [blame] [check-ins using] [diff]

This also removes #include <time.h> from OFHTTPClientTests, as this
would cause <time.h> to be included without _POSIX_C_SOURCE set to
199309L and thus struct timespec would be missing. file: [64b9648b5e] check-in: [bfc762c6d8] user: js, branch: trunk, size: 12795 [annotate] [blame] [check-ins using] [diff]

OS X needs this. file: [aa33d9d534] check-in: [e30269f87e] user: js, branch: trunk, size: 12785 [annotate] [blame] [check-ins using] [diff]

Not using -std=gnu11 means _GNU_SOURCE does not get defined anymore,
therefore this commit also adds the required feature defines for glibc.

Additionally, this adds of_strdup in macros.h, as strdup is an
extension. file: [f7a0f4a2c6] check-in: [2f5af58573] user: js, branch: trunk, size: 12759 [annotate] [blame] [check-ins using] [diff]

This is used instead for time intervals now instead of doubles. file: [8e1ed139e9] check-in: [dc31ce3d34] user: js, branch: trunk, size: 12756 [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: [34887ea375] check-in: [fa6496efc7] user: js, branch: trunk, size: 12618 [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: [d90c31fbc5] check-in: [3d16a30f41] user: js, branch: trunk, size: 12626 [annotate] [blame] [check-ins using] [diff]

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

If the argument is nil, it will return the receiver. file: [6b0927552c] check-in: [38dda0d1c7] user: js, branch: trunk, size: 13287 [annotate] [blame] [check-ins using] [diff]

This makes it clear that multiple evaluation of parameters is possible. file: [66b4bbf344] check-in: [1cb8fee5c3] user: js, branch: trunk, size: 13164 [annotate] [blame] [check-ins using] [diff]

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

printf's %a was too fragile to serialize floats and doubles, as it was
behaving differently on different OSes and OS versions. Instead, a hex
representation of the float/double in big endian is used now. While this
is less readable, it is guaranteed to be accurate and always the same. file: [1da3535d97] check-in: [a6ee2b2065] user: js, branch: trunk, size: 13112 [annotate] [blame] [check-ins using] [diff]

This is to be used when the condition includes a statement that needs to
be executed, as with assert, this would not be executed anymore when
NDEBUG is defined. file: [c0e29e0a91] check-in: [417b213d41] user: js, branch: trunk, size: 12926 [annotate] [blame] [check-ins using] [diff]

Only add an extra day for the current year if it is a leap year and it
is at least March. file: [7dd968f934] check-in: [5d102d382e] user: js, branch: trunk, size: 12644 [annotate] [blame] [check-ins using] [diff]

This way, we don't need tm_gmtoff, which we would otherwise need to
adjust the value parsed by mktime, and don't need timegm either. file: [396325df1e] check-in: [4fc7884fb1] user: js, branch: trunk, size: 12356 [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: [03b1ee2aba] check-in: [e1e7ffa903] user: js, branch: trunk, size: 11529 [annotate] [blame] [check-ins using] [diff]

nearbyint() is missing on some systems, even though it is part of C99
and POSIX. file: [a20bfc7333] check-in: [45b499fb11] user: js, branch: trunk, size: 11444 [annotate] [blame] [check-ins using] [diff]

Also fixes the forgotten change to double in +[sleepForTimeInterval:]
and fixes two warnings due to missing casts from double to uint32_t. file: [0cf451e330] check-in: [c19b2b5726] user: js, branch: trunk, size: 11444 [annotate] [blame] [check-ins using] [diff]

This way, we don't need a new parser and don't use a new format. file: [1c6620e225] check-in: [494dca4ae4] user: js, branch: trunk, size: 11847 [annotate] [blame] [check-ins using] [diff]

* Don't expose time_t to the user.
* Use better return types for -[minute] etc.
* Add +[dateWithTimeIntervalSinceNow:].
* Add -[dateByAddingTimeInterval:]. file: [a3f6814aa0] check-in: [c084907c2c] user: js, branch: trunk, size: 9153 [annotate] [blame] [check-ins using] [diff]