Major revamp. I've been trying to clean up the code. the bzerror stuff
authorAaron Lehmann <aaronl@vitelius.com>
Sat, 22 Jun 2002 06:35:29 +0000 (06:35 -0000)
committerAaron Lehmann <aaronl@vitelius.com>
Sat, 22 Jun 2002 06:35:29 +0000 (06:35 -0000)
commit9f92d5f2a26d0ef8cc8fd38a953512b82433e78a
tree194adec0296d00ba5b9676b2881e4aa748e5eba9
parent2d34215456def242968e5b545245d5d6ec4af432
Major revamp. I've been trying to clean up the code. the bzerror stuff
still needs major attention (i.e. removal).

Removed references to uninitialized variables like progName,
smallMode, noisy, etc. Remove functions and code for handling "small"
decompression mode, since it is all unreachable. Remove
total_{in,out}* counters - they are never used. Remove panic()
function and assert_h and their uses because they are all for "should
never happen" circumstances. Replace internal malloc/free wrappers
with xmalloc and free. Remove conditional in if(foo)free(foo);
situations. Remove bogus
if (sizeof(int) != 4) {
return BZ_CONFIG_ERROR;
}
(...etc...) code and code for handling BZ_CONFIG_ERROR. Someone should
go through and change the applet to use well-defined types when
appropriate - it expects sizeof(short)==2 && sizeof(int)==4. Until
this commit these were explicitly checked for, and the applet would
exit if these types were not the right size. (I think this is wrong
even as an interim solution.)

With gcc 3.1 reduces size of binary on i386-linux by about 700 bytes.

Lightly tested.
archival/bunzip2.c