RFR: JDK-8241883: (zipfs) SeekableByteChannel:close followed by SeekableByteChannel:close will throw an NPE

[Lance Andersen]

Thank you Alan.

I will push this back today.

Best
Lance

> On Apr 10, 2020, at 3:23 AM, Alan Bateman <Alan.Bateman at oracle.com> wrote:
> 
> On 09/04/2020 19:20, Lance Andersen wrote:
>> 
>> Of course, and can be found at: http://cr.openjdk.java.net/~lancea/8241883/webrev.00/index.html
>> 
>> I modified the patch to use the beginWrite() from ByteArrayChannel per Christoph’s suggestion (I had done that originally but talked myself out of that for some reason …)
> Thanks for this, the inline patch didn't look right because it didn't coordinate the locking with the super class but this version seems to do it right. So I think it looks good.
> 
> -Alan.

 <http://oracle.com/us/design/oracle-email-sig-198324.gif>
 <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
 <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering 
1 Network Drive 
Burlington, MA 01803
Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/nio-dev/attachments/20200410/df771a37/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: oracle_sig_logo.gif
Type: image/gif
Size: 658 bytes
Desc: not available
URL: <https://mail.openjdk.java.net/pipermail/nio-dev/attachments/20200410/df771a37/oracle_sig_logo.gif>