ObjFW  History of tests/OFArrayTests.m of 425762ff70e486e9

ObjFW's style changed over the years, but the tests were never adjusted
to it. file: [a384e7dc9e] check-in: [14f1e22d79] user: js, branch: trunk, size: 12816 [annotate] [blame] [check-ins using] [diff]

Since C guarantees minimum sizes for char, short, int, long and long
long, these can be used instead. file: [650205073d] check-in: [25c985fec1] user: js, branch: trunk, size: 11896 [annotate] [blame] [check-ins using] [diff]

OFAutoreleasePools have long been discouraged in favor of the runtime's
autorelease pools. file: [54def65eb2] check-in: [09be825f88] user: js, branch: trunk, size: 11897 [annotate] [blame] [check-ins using] [diff]

It's not very useful and is a problem for the Foundation bridge, as
NSEnumerator does not have it. file: [501a7abef0] check-in: [7214a2c7af] user: js, branch: trunk, size: 11899 [annotate] [blame] [check-ins using] [diff]

This prevents forgetting to add an import to ObjFW.h, as the tests would
then not compile. file: [d070876349] check-in: [dc0c6c34ab] user: js, branch: trunk, size: 11954 [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: [4fabdad1e7] check-in: [2f4e0df8be] user: js, branch: trunk, size: 12130 [annotate] [blame] [check-ins using] [diff]

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

These are usually not used, as OFArray_adjacent and
OFMutableArray_adjacent are used instead. However, they are used if
someone creates their own subclass of OFArray / OFMutableArray. As they
are rarely used in production, it is important to run all tests on them. file: [c99073b249] check-in: [7fec2ef57e] user: js, branch: trunk, size: 11970 [annotate] [blame] [check-ins using] [diff]

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

Contrary to what the documentation stated, it did not actually call
super when the key was starting with an @. file: [1774d6c94a] check-in: [134c19b100] user: js, branch: trunk, size: 10035 [annotate] [blame] [check-ins using] [diff]

If the key starts with an @, the @ is stripped and the super method is
called. Otherwise, this returns the value for all objects / sets the
value for all objects. file: [75808adfc5] check-in: [05196e511a] user: js, branch: trunk, size: 9913 [annotate] [blame] [check-ins using] [diff]

Also improves the test to prevent this from happening again. file: [ea65aa5e42] check-in: [8a14ad35aa] user: js, branch: trunk, size: 9081 [annotate] [blame] [check-ins using] [diff]

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

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

Now that we require GCC >= 4.6, those features are always available. file: [db4b803ef8] check-in: [be8db49635] user: js, branch: trunk, size: 9083 [annotate] [blame] [check-ins using] [diff]

This allows to skip empty components. file: [007d7f7189] check-in: [f975b722df] user: js, branch: trunk, size: 9142 [annotate] [blame] [check-ins using] [diff]

Clang now warns if the enumeration variable of a fast enumeration is not
used. file: [c4b233b4a6] check-in: [6143baaaee] user: js, branch: trunk, size: 8904 [annotate] [blame] [check-ins using] [diff]

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

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

For consistency with Foundation. file: [1073eae76a] check-in: [26f2ffa908] user: js, branch: trunk, size: 8896 [annotate] [blame] [check-ins using] [diff]

This is required for the new array literals. file: [19f0fe9122] check-in: [008be86a16] user: js, branch: trunk, size: 8884 [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: [723d9a0843] check-in: [e1e7ffa903] user: js, branch: trunk, size: 8249 [annotate] [blame] [check-ins using] [diff]

This only makes sense as we use "Object" in the method names, implying
it's only affecting one object. file: [895b4e152f] check-in: [de45535c23] user: js, branch: trunk, size: 6622 [annotate] [blame] [check-ins using] [diff]