emo/busybox
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

gzip: use "unsigned" type for bit fields and bit counts

Browse Source 
This does not change any logic, those values should always be positive.

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
This commit is contained in:
Denys Vlasenko 2018-01-30 23:53:38 +01:00
parent 6ba6a6f28e
commit d7500f856d
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
archival/gzip.c
View File

@ -363,7 +363,7 @@ struct globals {
/* Number of bits used within bi_buf. (bi_buf might be implemented on /* Number of bits used within bi_buf. (bi_buf might be implemented on
* more than 16 bits on some systems.) * more than 16 bits on some systems.)
*/ */
int bi_valid; unsigned bi_valid;
#ifdef DEBUG #ifdef DEBUG
ulg bits_sent; /* bit length of the compressed data */ 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. * Send a value on a given number of bits.
* IN assertion: length <= 16 and value fits in length 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; unsigned new_buf;
int remain; unsigned remain;
#ifdef DEBUG #ifdef DEBUG
Tracev((stderr, " l %2d v %4x ", length, value)); Tracev((stderr, " l %2d v %4x ", length, value));
@ -548,7 +548,7 @@ static void send_bits(int value, int length)
} else { /* 16 */ } else { /* 16 */
put_16bit(new_buf); put_16bit(new_buf);
} }
new_buf = (unsigned) value >> remain; new_buf = value >> remain;
length -= BUF_SIZE; length -= BUF_SIZE;
} }
G1.bi_buf = new_buf; G1.bi_buf = new_buf;