[7u10] Request for approval for CR 7188852 - Move implementation of De/Inflater.getBytesRead/Writtten() to java from native

[Andrew Hughes]

----- Original Message -----
> On 19/09/2012 20:01, Seán Coffey wrote:
> >
> > On 19/09/2012 18:22, Andrew Hughes wrote:
> >> Sure, http://cr.openjdk.java.net/~andrew/zlib/webrev.01/ is the
> >> combined webrev.
> >> http://hg.openjdk.java.net/jdk8/jdk8/jdk/rev/35e024c6a62c is the
> >> original
> >> jdk8 changeset for 7110151.
> >
> > Thanks - let's get approval from Alan/Sherman then. I think Sherman
> > may be OOTO at the moment. Once they're ok, I'll log a request for
> > you
> > to integrate into 7u10 forests (once available).
> Yes, Sherman is away this week. I looked through Andrew's combined
> webrev and it looks good to me. To my knowledge, we haven't had any
> issues with these changes in jdk8. It seems a fine change to get into
> jdk7u.
> 

Are we still waiting for Sherman on this, or can I push?

> -Alan
> 

-- 
Andrew :)

Free Java Software Engineer
Red Hat, Inc. (http://www.redhat.com)

PGP Key: 248BDC07 (https://keys.indymedia.org/)
Fingerprint = EC5A 1F5E C0AD 1D15 8F1F  8F91 3B96 A578 248B DC07