[kaffe] patch for GZIPOutputStream

Dalibor Topic robilad@kaffe.org
Fri May 21 12:55:24 2004


Hi Jrandom,

jrandom wrote:
> To make up for the poor variable naming in my last post
> (s/mediaType/parameter/g), here's a minipatch for GZIPOutputStream:
> 
> Index: GZIPOutputStream.java
> ===================================================================
> RCS file:
> /cvs/kaffe/kaffe/libraries/javalib/java/util/zip/GZIPOutputStream.java,v
> retrieving revision 1.6
> diff -u -r1.6 GZIPOutputStream.java
> --- GZIPOutputStream.java       21 Apr 2004 14:59:06 -0000      1.6
> +++ GZIPOutputStream.java       5 May 2004 22:47:09 -0000
> @@ -108,6 +108,7 @@
> 
>      out.write(gzipHeader);
>      //    System.err.println("wrote GZIP header (" + gzipHeader.length +
> " bytes )");
> +    crc = new CRC32();
>    }
> 
>    public synchronized void write(byte[] buf, int off, int len)
> 
> (crc was otherwise never initialized, causing NPE on .write)

Thanks for your patch, the issue has been fixed in GNU Classpath, and 
I've merged that code over.

cheers,
dalibor topic