Re: RFR: 8282648: Problems due to conflicting specification of Inflater::inflate(..) and InflaterInputStream::read(..) [v4]

2022-04-11 Thread Volker Simonis
On Fri, 8 Apr 2022 13:21:06 GMT, Jaikiran Pai wrote: >> Volker Simonis has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Added API-refinement to GZIPInputStream::read()/ZipInputStream::read() and >> an Implementation note to ZipFile::getI

Re: RFR: 8282648: Problems due to conflicting specification of Inflater::inflate(..) and InflaterInputStream::read(..) [v4]

2022-04-11 Thread Volker Simonis
On Fri, 8 Apr 2022 13:11:23 GMT, Jaikiran Pai wrote: >> Volker Simonis has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Added API-refinement to GZIPInputStream::read()/ZipInputStream::read() and >> an Implementation note to ZipFile::getI

Re: RFR: 8282648: Problems due to conflicting specification of Inflater::inflate(..) and InflaterInputStream::read(..) [v4]

2022-04-08 Thread Jaikiran Pai
On Wed, 6 Apr 2022 16:28:21 GMT, Volker Simonis wrote: >> Add an API note to `InflaterInputStream::read(byte[] b, int off, int len)` >> to highlight that it might write more bytes than the returned number of >> inflated bytes into the buffer `b`. >> >> The superclass `java.io.InputStream` spe

Re: RFR: 8282648: Problems due to conflicting specification of Inflater::inflate(..) and InflaterInputStream::read(..) [v4]

2022-04-08 Thread Jaikiran Pai
On Wed, 6 Apr 2022 16:28:21 GMT, Volker Simonis wrote: >> Add an API note to `InflaterInputStream::read(byte[] b, int off, int len)` >> to highlight that it might write more bytes than the returned number of >> inflated bytes into the buffer `b`. >> >> The superclass `java.io.InputStream` spe

Re: RFR: 8282648: Problems due to conflicting specification of Inflater::inflate(..) and InflaterInputStream::read(..) [v4]

2022-04-08 Thread Jaikiran Pai
On Wed, 6 Apr 2022 16:28:21 GMT, Volker Simonis wrote: >> Add an API note to `InflaterInputStream::read(byte[] b, int off, int len)` >> to highlight that it might write more bytes than the returned number of >> inflated bytes into the buffer `b`. >> >> The superclass `java.io.InputStream` spe

Re: RFR: 8282648: Problems due to conflicting specification of Inflater::inflate(..) and InflaterInputStream::read(..) [v4]

2022-04-06 Thread Volker Simonis
> Add an API note to `InflaterInputStream::read(byte[] b, int off, int len)` to > highlight that it might write more bytes than the returned number of > inflated bytes into the buffer `b`. > > The superclass `java.io.InputStream` specifies that `read(byte[] b, int off, > int len)` will leave t