Thanks! This change looks very good to me - why did we the maintainers of zip_util never notice it? It's worth a later backport to jdk7 as well.
On Tue, Jan 8, 2013 at 10:53 AM, Xueming Shen <xueming.s...@oracle.com>wrote: > The correct list is core-libs-dev@openjdk.java.net > > I will be the sponsor for this change. > > -Sherman > > > On 01/08/2013 10:38 AM, Coleen Phillimore wrote: > >> >> Hi Ioi, >> >> I copied it to >> cr.openjdk.java.net/~coleenp/**8005466_zip_util_001<http://cr.openjdk.java.net/~coleenp/8005466_zip_util_001> >> >> I think the mailing list is jdk8-...@openjdk.java.net. I don't know >> about jdk8. Since it's a tools/libraries fix, it should be relative to >> the TL repository (not sure if your fix is). >> >> I'm apparently "reviewer" on jdk8 and allowed to sponsor changes to jdk8 >> but I'd prefer if someone like Alan Bateman who actually works on the jdk >> do this. >> >> This fix seems cool. >> >> Thanks, >> Coleen >> >> On 01/08/2013 11:43 AM, Ioi Lam wrote: >> >>> Coleen, >>> >>> https://jbs.oracle.com/bugs/**browse/JDK-8005466?** >>> focusedCommentId=13288645#**comment-13288645<https://jbs.oracle.com/bugs/browse/JDK-8005466?focusedCommentId=13288645#comment-13288645> >>> http://javaweb.us.oracle.com/~**iklam/webrev/8005466_zip_util_**001/<http://javaweb.us.oracle.com/~iklam/webrev/8005466_zip_util_001/>< >>> http://javaweb.us.oracle.com/**%7Eiklam/webrev/8005466_zip_**util_001/<http://javaweb.us.oracle.com/%7Eiklam/webrev/8005466_zip_util_001/> >>> > >>> >>> A one-liner would save about 540KB with Eclipse on 64-bit VMs. >>> >>> BTW, which mailing list should this be sent to? hotspot-dev@openjdk? >>> >>> Karen, would it be possible for this to get into JDK8? >>> >>> Thanks >>> - Ioi >>> >> >> >
