ObjFW  History of src/OFDeflateStream.m of bd2389b6b88adc02

There is not much point in having a bi-directional stream for Deflate
compression / decompression. check-in: [f56a50ee50] user: js, branch: trunk, size: 0

Instead, let reads return 0 and let writes append after the end. file: [f38e0e21e0] check-in: [f9cd4f9cab] user: js, branch: trunk, size: 20257 [annotate] [blame] [check-ins using] [diff]

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

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

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

Returning true if _decompression is NULL is a bad idea, as it means
-[isAtEndOfStream] will return true when no read has happened yet. This
breaks the common case where -[isAtEndOfStream] is checked in the loop
condition and the read only happens in the loop body. file: [c4f1ea9fcd] check-in: [8e4451aee2] user: js, branch: trunk, size: 20150 [annotate] [blame] [check-ins using] [diff]

The reason for renaming to OFInflateStream was to have one stream for
decompression and one for compression in order to reduce memory usage if
only one of the two is needed, as the ivar layout will be smaller then.
However, it is more consistent with other stream classes to have one
stream that can handle both. The increased memory footprint of having
ivars for compression and decompression can be solved by having a
pointer to those instead. This will not incur any performance penalty,
as the pointer will be dereferenced instead of the ivars, meaning the
overhead is only getting the pointer from the ivars once. file: [7f4785aa79] check-in: [134a1121c7] user: js, branch: trunk, size: 19216 [annotate] [blame] [check-ins using]

It makes more sense to have one class for inflating and one for
deflating instead of having one that does both, as having one that does
both requires more memory per instance when only inflating or deflating
is used. check-in: [2e9336b1ef] user: js, branch: trunk, size: 0

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: [505e08ef0e] check-in: [13ee56edf3] user: js, branch: trunk, size: 18852 [annotate] [blame] [check-ins using] [diff]

This replaces -[numberOfBytesInReadBuffer], as it's not always known how
many bytes there are exactly and thus the number would often be wrong
(e.g. because data is compressed). file: [e0f523d409] check-in: [acc999a75e] user: js, branch: trunk, size: 18896 [annotate] [blame] [check-ins using] [diff]

This is in preparation for adding UDP sockets, as UDP sockets and TCP
sockets have no common superclass, as one is stream-oriented while the
other is packet-oriented.

Read and write exceptions are for any object now, as they are useful for
a lot more than just for streams, while the others (bind, listen, etc.)
are for any socket now (the type is id in this case, though, as there is
no common superclass). file: [f380a0e937] check-in: [8d2a5052fd] user: js, branch: trunk, size: 18085 [annotate] [blame] [check-ins using] [diff]

In case the uncompressed header was incomplete, this overwrote the old
buffer instead of continuing it. file: [b45e69217a] check-in: [d1293b647a] user: js, branch: trunk, size: 17473 [annotate] [blame] [check-ins using] [diff]

When a bit length had more than 255 occurrences, it caused an integer
overflow and thus a corrupt tree.

This also cleans up tree walking. file: [a8810ba694] check-in: [aee3bc12f7] user: js, branch: trunk, size: 17452 [annotate] [blame] [check-ins using] [diff]

On systems where uint_fast16_t is different from uint16_t, the pointer
operator creates incompatible pointers for those two, thus leading to a
compile time error. Therefore, one uint_fast16_t needed to be changed to
an uint16_t. file: [d790744286] check-in: [5dbf9298bd] user: js, branch: trunk, size: 17640 [annotate] [blame] [check-ins using] [diff]

No compression support yet, only decompression.

Decompression speed is acceptable for productive use, but there is still
a lot of room for optimization as this is a very straightforward
implementation without much optimization. file: [856d77bd39] check-in: [d83d3aa719] user: js, branch: trunk, size: 17650 [annotate] [blame] [check-ins using]