ObjFW  History of src/OFTarArchive.m of 2cb48ceaf1d0c54a

Some names are too generic and might cause conflicts (e.g. zoo). file: [1855b6b9b8] check-in: [13f0321eff] user: js, branch: trunk, size: 11241 [annotate] [blame] [check-ins using] [diff]

This allows for a much cleaner solution to avoid the internal read
buffer of e.g. a TLS connection never being processed while waiting for
a delimiter. file: [1c76370f81] check-in: [a61e0594b4] user: js, branch: trunk, size: 11193 [annotate] [blame] [check-ins using] [diff]

It's private anyway. file: [734cc4a931] check-in: [e520af4113] user: js, branch: trunk, size: 11216 [annotate] [blame] [check-ins using] [diff]

It's generally uncommon to prefix those.

Additionally, it might be worthwhile to try to get those actually
registered, which requires usage. file: [e86038baec] check-in: [9b3cae6cba] user: js, branch: trunk, size: 10967 [annotate] [blame] [check-ins using] [diff]

In order to not create a retain cycle, this changes the reference to the
last returned stream to an unsafe unretained one that the stream itself
resets to nil in its dealloc. file: [d33bd24f84] check-in: [6c08b57605] user: js, branch: trunk, size: 10921 [annotate] [blame] [check-ins using] [diff]

The old write API made it too easy to lose bytes when a stream is set to
non-blocking mode. The new API always throws when not all bytes were
written, which forces handling the number of bytes being written being
smaller than the number of bytes requested to be written. file: [651ec2f405] check-in: [d1d36ae522] user: js, branch: trunk, size: 10235 [annotate] [blame] [check-ins using] [diff]

Still miscompiles things. file: [6889b3aa0d] check-in: [9e9ce6aa1c] user: js, branch: trunk, size: 10497 [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: [9fd5fb5c63] check-in: [3b43d51006] user: js, branch: trunk, size: 10273 [annotate] [blame] [check-ins using] [diff]

This is currently limited to reading uncompressed LHA level 2 files. file: [d5ac892200] check-in: [790b5344d6] user: js, branch: trunk, size: 9874 [annotate] [blame] [check-ins using] [diff]

Instead of letting OFStream conform to it, which itself does not really
conform to it, let all the subclasses that actually do conform to it. file: [247f54af17] check-in: [69749b6a5b] user: js, branch: trunk, size: 9581 [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: [deb3b00a60] check-in: [2f4e0df8be] user: js, branch: trunk, size: 9442 [annotate] [blame] [check-ins using] [diff]

It does not work well enough yet: While creating a lot of noise, it
fails to even find many of the obvious cases. Depending on the static
analyzer seems to be the better approach for now. file: [5865d323f0] check-in: [e47c219327] user: js, branch: trunk, size: 9349 [annotate] [blame] [check-ins using] [diff]

If the stream is seekable, seek instead of reading to skip. file: [944c9a1051] check-in: [29ee511f56] user: js, branch: trunk, size: 9156 [annotate] [blame] [check-ins using] [diff]

This is now only necessary for reading and appending. file: [cb4c96888d] check-in: [c404c33cf1] user: js, branch: trunk, size: 10365 [annotate] [blame] [check-ins using] [diff]

This is in preparation for OFTarArchive write support. file: [0db6ddc7de] check-in: [3f34a360bf] user: js, branch: trunk, size: 4838 [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: [4c3a4649ab] check-in: [23e57c5040] user: js, branch: trunk, size: 2366 [annotate] [blame] [check-ins using] [diff]

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

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

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

This only implements the UNIX v7 format for now. file: [370eb5f8c8] check-in: [7ecdc71677] user: js, branch: trunk, size: 2373 [annotate] [blame] [check-ins using]