updated review request for 7021582, use try-with-resources in jar/zip implementation and tests

Alan Bateman Alan.Bateman at oracle.com
Fri Feb 25 10:20:16 UTC 2011


Stuart Marks wrote:
> Hi all, updated webrev is here:
>
> http://cr.openjdk.java.net/~smarks/reviews/7021582/webrev.1/
>
> I've removed the pack200 files (i.e. those in 
> src/share/classes/com/sun/java/util/jar/pack) so I can do more 
> thorough deploy/install testing per Kumar's request. The change to 
> PropMap.java, which generated a fair bit of discussion, is actually in 
> the pack200 library so it's no longer part of this webrev.
These changes look good to me.

-Alan.



More information about the core-libs-dev mailing list