PatchSet 4591 Date: 2004/03/29 20:30:56 Author: dalibor Branch: HEAD Tag: (none) Log: Fixed all sign-compare-1 warnings
Members: ChangeLog:1.2169->1.2170 kaffe/kaffevm/inflate.c:1.11->1.12 kaffe/kaffevm/inflate.h:1.2->1.3 Index: kaffe/ChangeLog diff -u kaffe/ChangeLog:1.2169 kaffe/ChangeLog:1.2170 --- kaffe/ChangeLog:1.2169 Mon Mar 29 20:24:26 2004 +++ kaffe/ChangeLog Mon Mar 29 20:30:56 2004 @@ -1,5 +1,11 @@ 2004-03-29 Adam Heath <[EMAIL PROTECTED]> + * kaffe/kaffevm/inflate.c + kaffe/kaffevm/inflate.h: + Fixed all sign-compare-1 warnings. + +2004-03-29 Adam Heath <[EMAIL PROTECTED]> + First pass at adding __attribute__ to various places. 2004-03-29 Adam Heath <[EMAIL PROTECTED]> Index: kaffe/kaffe/kaffevm/inflate.c diff -u kaffe/kaffe/kaffevm/inflate.c:1.11 kaffe/kaffe/kaffevm/inflate.c:1.12 --- kaffe/kaffe/kaffevm/inflate.c:1.11 Mon Sep 22 15:31:24 2003 +++ kaffe/kaffe/kaffevm/inflate.c Mon Mar 29 20:30:57 2004 @@ -33,14 +33,14 @@ } v; } huft; -static int inflate_codes(inflateInfo*, huft*, huft*, int, int); +static int inflate_codes(inflateInfo*, huft*, huft*, size_t, size_t); static int inflate_stored(inflateInfo*); static int inflate_fixed(inflateInfo*); static int inflate_dynamic(inflateInfo*); static int inflate_block(inflateInfo*, int*); static int huft_free(huft*); static int inflate(inflateInfo*); -static int huft_build(inflateInfo* pG, unsigned* b, unsigned n, unsigned s, uint16* d, uint16* e, huft** t, int* m); +static int huft_build(inflateInfo* pG, unsigned* b, unsigned n, unsigned s, uint16* d, uint16* e, huft** t, size_t* m); /* Tables for deflate from PKZIP's appnote.txt. */ static unsigned border[] = { /* Order of the bit length code lengths */ @@ -109,15 +109,15 @@ static int -inflate_codes(inflateInfo* pG, huft* tl, huft* td, int bl, int bd) +inflate_codes(inflateInfo* pG, huft* tl, huft* td, size_t bl, size_t bd) { - register unsigned e; /* table entry flag/number of extra bits */ - unsigned n, d; /* length and index for copy */ - unsigned w; /* current window position */ + register size_t e; /* table entry flag/number of extra bits */ + size_t n, d; /* length and index for copy */ + size_t w; /* current window position */ huft *t; /* pointer to table entry */ - unsigned ml, md; /* masks for bl and bd bits */ + int ml, md; /* masks for bl and bd bits */ register uint32 b; /* bit buffer */ - register unsigned k; /* number of bits in bit buffer */ + register size_t k; /* number of bits in bit buffer */ /* make local copies of globals */ @@ -131,7 +131,7 @@ md = mask_bits[bd]; while (1) /* do until end of block */ { - NEEDBITS(pG, (unsigned)bl) + NEEDBITS(pG, bl) if ((e = (t = tl + ((unsigned)b & ml))->e) > 16) do { if (e == 99) @@ -162,7 +162,7 @@ DUMPBITS(pG, e); /* decode distance of block to copy */ - NEEDBITS(pG, (unsigned)bd) + NEEDBITS(pG, bd) if ((e = (t = td + ((unsigned)b & md))->e) > 16) do { if (e == 99) @@ -213,10 +213,10 @@ int inflate_stored(inflateInfo* pG) { - unsigned n; /* number of bytes in block */ - unsigned w; /* current window position */ - register uint32 b; /* bit buffer */ - register unsigned k; /* number of bits in bit buffer */ + size_t n; /* number of bytes in block */ + size_t w; /* current window position */ + register int32 b; /* bit buffer */ + register int k; /* number of bits in bit buffer */ /* make local copies of globals */ @@ -320,8 +320,8 @@ unsigned n; /* number of lengths to get */ huft *tl; /* literal/length code table */ huft *td; /* distance code table */ - int bl; /* lookup bits for tl */ - int bd; /* lookup bits for td */ + size_t bl; /* lookup bits for tl */ + size_t bd; /* lookup bits for td */ unsigned nb; /* number of bit length codes */ unsigned nl; /* number of literal/length codes */ unsigned nd; /* number of distance codes */ @@ -630,17 +630,17 @@ static int -huft_build(inflateInfo* pG, unsigned* b, unsigned n, unsigned s, uint16* d, uint16* e, huft** t, int* m) +huft_build(inflateInfo* pG, unsigned* b, unsigned n, unsigned s, uint16* d, uint16* e, huft** t, size_t* m) { unsigned a; /* counter for codes of length k */ unsigned c[BMAX+1]; /* bit length count table */ unsigned el; /* length of EOB code (value 256) */ unsigned f; /* i repeats in table every f entries */ - int g; /* maximum code length */ + size_t g; /* maximum code length */ int h; /* table level */ register unsigned i; /* counter, current code */ register unsigned j; /* counter */ - register int k; /* number of bits in current code */ + register size_t k; /* number of bits in current code */ int lx[BMAX+1]; /* memory for l[-1..BMAX-1] */ int *l = lx+1; /* stack of bits per table */ register unsigned *p; /* pointer into c[], b[], or v[] */ @@ -648,11 +648,11 @@ huft r; /* table entry for structure assignment */ huft *u[BMAX]; /* table stack */ unsigned v[N_MAX]; /* values in order of bit length */ - register int w; /* bits before this table == (l * h) */ + register size_t w; /* bits before this table == (l * h) */ unsigned x[BMAX+1]; /* bit offsets, then code stack */ unsigned *xp; /* pointer into x */ int y; /* number of dummy codes added */ - unsigned z; /* number of entries in current table */ + size_t z; /* number of entries in current table */ /* Generate counts for each bit length */ @@ -734,7 +734,7 @@ w += l[h++]; /* add bits already decoded */ /* compute minimum size table less than or equal to *m bits */ - z = (z = g - w) > (unsigned)*m ? *m : z; /* upper limit */ + z = (z = g - w) > *m ? *m : z; /* upper limit */ if ((f = 1 << (j = k - w)) > a + 1) /* try a k-w bit table */ { /* too few codes for k-w bit table */ f -= a + 1; /* deduct codes from patterns left */ Index: kaffe/kaffe/kaffevm/inflate.h diff -u kaffe/kaffe/kaffevm/inflate.h:1.2 kaffe/kaffe/kaffevm/inflate.h:1.3 --- kaffe/kaffe/kaffevm/inflate.h:1.2 Sun May 31 00:08:36 1998 +++ kaffe/kaffe/kaffevm/inflate.h Mon Mar 29 20:30:58 2004 @@ -15,6 +15,7 @@ #ifndef __inflate_h #define __inflate_h +#include <sys/types.h> struct _huft; typedef struct _inflateInfo { @@ -22,15 +23,15 @@ unsigned hufts; /* track memory usage */ struct _huft* fixed_tl; struct _huft* fixed_td; - int fixed_bl; - int fixed_bd; + size_t fixed_bl; + size_t fixed_bd; unsigned wp; /* current position in slide */ uint32 bb; /* bit buffer */ unsigned bk; /* bits in bit buffer */ uint8* inbuf; - int insz; + size_t insz; uint8* outbuf; - int outsz; + size_t outsz; } inflateInfo; extern inflateInfo* inflate_new(void); _______________________________________________ kaffe mailing list [EMAIL PROTECTED] http://kaffe.org/cgi-bin/mailman/listinfo/kaffe