[JDK-6341887] RFR: Patch V3: java.util.zip: Add ByteBuffer methods to Inflater/Deflater

[David Lloyd]

OK thanks for the update.  It seems odd to me - C99 has been around
for a pretty long time - but, as long as the change goes in, I'm
happy.  Thanks!

On Thu, Apr 19, 2018 at 1:13 AM, Xueming Shen <xueming.shen at oracle.com> wrote:
> David,
>
> webrev has been updated to address the compiler error on solaris-sparcv9.
> The C comipler on
> solaris-sparcv9 platform does not like those "declaration follow a
> statement" new code in
> Deflater.c and Inflater.c
>
> http://cr.openjdk.java.net/~sherman/6341887.David.Lloyd/webrev/
>
> Thanks,
> Sherman
>
>
> On 4/18/18, 1:16 PM, David Lloyd wrote:
>
> +1 from me, thanks!
>
> On Wed, Apr 18, 2018 at 3:09 PM, Xueming Shen <xueming.shen at oracle.com>
> wrote:
>
> Alan, David,
>
> Any more update/comment/suggestion on this one? I have updated
> DeInflate.java with
> some new test cases to cover the newly added methods. Good to go?
>
> -Sherman
>
> On 04/10/2018 08:49 PM, Xueming Shen wrote:
>
>
> http://cr.openjdk.java.net/~sherman/6341887.David.Lloyd/webrev/
>
> Thanks,
> Sherman
>
>
>
>



-- 
- DML