ObjFW  History of src/OFBlock.m of 50dce32144850e5f

errno is problematic for Amiga libraries and is also not thread-safe on
some systems, even though it should. file: [ada45ac629] check-in: [5b37fbeb82] user: js, branch: trunk, size: 12179 [annotate] [blame] [check-ins using] [diff]

This allows the runtime to only link against the parts it needs, without
pulling in unnecessary parts like thread spawning, TLS and conditions. file: [acebb38475] check-in: [5358e9ea6a] user: js, branch: trunk, size: 12624 [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: [a2a5089265] check-in: [2f4e0df8be] user: js, branch: trunk, size: 12539 [annotate] [blame] [check-ins using] [diff]

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

This is necessary for ObjC++ with GCC, as otherwise, this generates an
error. Since the objc_abi_module struct is private anyway, there is not
much reason to have the argument of that type. file: [067d5d407d] check-in: [4e40a57efa] user: js, branch: trunk, size: 12431 [annotate] [blame] [check-ins using] [diff]

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

While at it, also make things thread-safe. file: [0d239bd3d6] check-in: [9dc0755178] user: js, branch: trunk, size: 12528 [annotate] [blame] [check-ins using] [diff]

Setting it before the memcpy() meant it was effectively a nop.

This also improves on the tests to catch this in the future. file: [ec9702862e] check-in: [dbb706d21e] user: js, branch: trunk, size: 11160 [annotate] [blame] [check-ins using] [diff]

While this makes moving things around easier, this has been done by
accident and looks ugly. file: [ce186b9740] check-in: [3572e824c6] user: js, branch: trunk, size: 11216 [annotate] [blame] [check-ins using] [diff]

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

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

The first was that _Block_object_{assign,dispose} would only be a NOP
when called from a byref helper instead of only being a NOP when called
for an object from a byref helper.

The second was that byref helpers would even be called when there are no
byref helpers (bit 25 in the byref flags needs to be checked -
conveniently, bit 25 indicates copy/dispose helpers in the block literal
flags, too, so that define can be reused). file: [536f7c35b9] check-in: [3c3467423a] user: js, branch: trunk, size: 11213 [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: [9cf137c0dc] check-in: [13ee56edf3] user: js, branch: trunk, size: 11217 [annotate] [blame] [check-ins using] [diff]

OS X/PPC64 is limited to 10.5 and thus it does not exist there, even
though __OBJC2__ is defined. file: [aa21105e0f] check-in: [3204ceeca0] user: js, branch: trunk, size: 11311 [annotate] [blame] [check-ins using] [diff]

As most blocks are static instances, there's a chance +[initialize] is
never called.

As a nice side effect, this also fixes the bug that the spinlocks were
initialized multiple times. file: [84017da3fd] check-in: [8fbc6b4e63] user: js, branch: trunk, size: 11503 [annotate] [blame] [check-ins using] [diff]

This way, we don't need the mach_alias_list anymore. file: [90b4a8e14a] check-in: [2bedfac177] user: js, branch: trunk, size: 11835 [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: [a204e2796b] check-in: [417b213d41] user: js, branch: trunk, size: 11934 [annotate] [blame] [check-ins using] [diff]

* master: (6 commits)
Improve -[OFNumber description].
... file: [d9ed74f314] check-in: [19a677b596] user: js, branch: runtime, size: 11689 [annotate] [blame] [check-ins using] [diff]

A linked-list is put before each memory chunk allocated instead of
having an array of all memory chunks. This means only one malloc now
instead of one malloc and one realloc. This also means that when
checking reallocs and frees, it's no longer necessary to iterate through
all memory chunks, as the linked list also contains the owner, meaning
realloc and free are no longer O(n), but O(1) now.

As allocating bigger chunks seems to be a little bit slower than smaller
chunks, it seems that this is slightly slower in benchmarks if only very small
chunks are allocated. However, measuring real world usage, it's a lot faster. file: [5825d5d284] check-in: [28170f5f65] user: js, branch: trunk, size: 12252 [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: [4b700a1d09] check-in: [e1e7ffa903] user: js, branch: trunk, size: 12361 [annotate] [blame] [check-ins using] [diff]

Hopefully, there is no weird situation left where it fails... file: [52c8c831cc] check-in: [7caa622124] user: js, branch: 0.5, size: 11547 [annotate] [blame] [check-ins using] [diff]

Hopefully, there is no weird situation left where it fails... file: [0147efccdf] check-in: [dbd79d0747] user: js, branch: trunk, size: 12271 [annotate] [blame] [check-ins using] [diff]

Currently, the properties test for nonatomic fails. This is a bug in
the new GNU runtime. file: [49b08c57ae] check-in: [6f4eb004a3] user: js, branch: trunk, size: 9781 [annotate] [blame] [check-ins using] [diff]

The reason is that the _Block_copy from libSystem references the wrong
_NSConcreteMallocBlock. file: [86829d5175] check-in: [4285aea3c6] user: js, branch: trunk, size: 9283 [annotate] [blame] [check-ins using] [diff]

The reason is that a block might be an NSBlock on OS X where it would
add the block to an NSAutoreleasePool. This change forces the developer
to use [OFAutoreleasePool addObject: block], which will also work if
the block is an NSBlock. Thus, this change is to force developers to
write portable code. file: [995214f475] check-in: [f78e4e4714] user: js, branch: trunk, size: 5408 [annotate] [blame] [check-ins using] [diff]

It seems it generates a conflict with some gcc versions. file: [0ee627944f] check-in: [eac76c7a30] user: js, branch: trunk, size: 5401 [annotate] [blame] [check-ins using] [diff]