From d7500f856d856716fd228935bb5e84c897c9daa8 Mon Sep 17 00:00:00 2001 From: Denys Vlasenko Date: Tue, 30 Jan 2018 23:53:38 +0100 Subject: [PATCH] gzip: use "unsigned" type for bit fields and bit counts This does not change any logic, those values should always be positive. Signed-off-by: Denys Vlasenko --- archival/gzip.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/archival/gzip.c b/archival/gzip.c index 4a3fe976a..f253a217a 100644 --- a/archival/gzip.c +++ b/archival/gzip.c @@ -363,7 +363,7 @@ struct globals { /* Number of bits used within bi_buf. (bi_buf might be implemented on * more than 16 bits on some systems.) */ - int bi_valid; + unsigned bi_valid; #ifdef DEBUG ulg bits_sent; /* bit length of the compressed data */ @@ -520,10 +520,10 @@ static unsigned file_read(void *buf, unsigned size) * Send a value on a given number of bits. * IN assertion: length <= 16 and value fits in length bits. */ -static void send_bits(int value, int length) +static void send_bits(unsigned value, unsigned length) { unsigned new_buf; - int remain; + unsigned remain; #ifdef DEBUG Tracev((stderr, " l %2d v %4x ", length, value)); @@ -548,7 +548,7 @@ static void send_bits(int value, int length) } else { /* 16 */ put_16bit(new_buf); } - new_buf = (unsigned) value >> remain; + new_buf = value >> remain; length -= BUF_SIZE; } G1.bi_buf = new_buf;