summaryrefslogtreecommitdiff
path: root/archival
diff options
context:
space:
mode:
authorDenys Vlasenko2018-01-30 22:37:06 +0100
committerDenys Vlasenko2018-01-30 22:37:06 +0100
commite4c4e6ddc365d8e9b5409e2f929116624c932c1d (patch)
tree446538d917e662f9cd986fd6507f3033b580aec2 /archival
parent9c499a5af48bd4d9d635c8021e6ad14fd4e939dc (diff)
downloadbusybox-e4c4e6ddc365d8e9b5409e2f929116624c932c1d.zip
busybox-e4c4e6ddc365d8e9b5409e2f929116624c932c1d.tar.gz
gzip: code shrink
function old new delta send_bits 92 70 -22 Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'archival')
-rw-r--r--archival/gzip.c31
1 files changed, 18 insertions, 13 deletions
diff --git a/archival/gzip.c b/archival/gzip.c
index 8ef6639..efc5a0e 100644
--- a/archival/gzip.c
+++ b/archival/gzip.c
@@ -358,7 +358,7 @@ struct globals {
unsigned short bi_buf;
#undef BUF_SIZE
-#define BUF_SIZE (8 * sizeof(G1.bi_buf))
+#define BUF_SIZE (int)(8 * sizeof(G1.bi_buf))
/* Number of bits used within bi_buf. (bi_buf might be implemented on
* more than 16 bits on some systems.)
@@ -522,24 +522,29 @@ static unsigned file_read(void *buf, unsigned size)
*/
static void send_bits(int value, int length)
{
+ unsigned new_buf;
+ int remain;
+
#ifdef DEBUG
Tracev((stderr, " l %2d v %4x ", length, value));
Assert(length > 0 && length <= 15, "invalid length");
G1.bits_sent += length;
#endif
- /* If not enough room in bi_buf, use (valid) bits from bi_buf and
- * (16 - bi_valid) bits from value, leaving (width - (16-bi_valid))
- * unused bits in value.
- */
- if (G1.bi_valid > (int) BUF_SIZE - length) {
- G1.bi_buf |= (value << G1.bi_valid);
- put_16bit(G1.bi_buf);
- G1.bi_buf = (ush) value >> (BUF_SIZE - G1.bi_valid);
- G1.bi_valid += length - BUF_SIZE;
- } else {
- G1.bi_buf |= value << G1.bi_valid;
- G1.bi_valid += length;
+
+ new_buf = G1.bi_buf | (value << G1.bi_valid);
+ remain = BUF_SIZE - G1.bi_valid;
+ /* If not enough room in bi_buf */
+ if (length > remain) {
+ /* ...use (valid) bits from bi_buf and
+ * (16 - bi_valid) bits from value,
+ * leaving (width - (16-bi_valid)) unused bits in value.
+ */
+ put_16bit(new_buf);
+ new_buf = (ush) value >> remain;
+ length -= BUF_SIZE;
}
+ G1.bi_buf = new_buf;
+ G1.bi_valid += length;
}