6989471: compiler warnings building java/zip native code

[Alan Bateman]

Xueming Shen wrote:
> Alan, Kelly,
>
> Would you please help review the patch that tries to address those 
> compiler warning
> in zip area?
>
> http://cr.openjdk.java.net/~sherman/6989471/webrev 
> <http://cr.openjdk.java.net/%7Esherman/6989471/webrev/>
>
> I added some comments to document the fact that 
> ZIP_Read/FindEntry/InflateFully can't
> deal with > 2**32 byte (the only user of these native method is the 
> vm, I don't think they
> are trying to de-compress a > 4G entry in one invocation any time 
> soon). We might find
> interesting to support > 2**32 entry for those methods, but obviously 
> it is not the purpose
> of this bug.
>
> Same for the "dstLen" of the compress/uncompress methods, the original 
> zlib interface
> has the limitation of 2**32. I only tried to remove the warning in 
> this patch.
>
> I'm not dealing with the POSIX strdup warning.
>
> It appears Kumar has already address the warning in zcrc32.c.
>
> Thanks,
> Sherman
In zip_util.c there is a comment that reads "reading A entry". Is that a 
typo that should be "reading an entry". Also looks like an indent 
problem at L1377.

Otherwise looks fine to me.

-Alan.



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/core-libs-dev/attachments/20101119/0a261eac/attachment.html>