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

[Andrew Hughes]

----- Original Message -----
> Alan has indicated that he's ok with this fix going into jdk7u.
> You're
> free to push to jdk7u-dev.

Done:

http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/c241d4a80272
http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/fc52d0a1fcda

Thanks.

> 
> Regards,
> Sean.
> 
> On 24/09/2012 14:24, Andrew Hughes wrote:
> >
> > ----- 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