Re: RFR: 8321053: Use ByteArrayInputStream.buf directly when parameter of transferTo() is trusted [v4]

2023-12-05 Thread Brian Burkhalter
On Tue, 5 Dec 2023 12:58:45 GMT, Markus KARG wrote: >> I suspect it's left over from a previous iteration. In any case, limiting it >> to a small number of output streams makes this easier to look at. BAOS and >> FOS seem okay, POP seems okay too but legacy and not interesting. > >> I suspect

Re: RFR: 8321053: Use ByteArrayInputStream.buf directly when parameter of transferTo() is trusted [v4]

2023-12-05 Thread Markus KARG
On Tue, 5 Dec 2023 08:27:09 GMT, Alan Bateman wrote: > I suspect it's left over from a previous iteration. In any case, limiting it > to a small number of output streams makes this easier to look at. BAOS and > FOS seem okay, POP seems okay too but legacy and not interesting. Agreed for a

Re: RFR: 8321053: Use ByteArrayInputStream.buf directly when parameter of transferTo() is trusted [v4]

2023-12-05 Thread Alan Bateman
On Tue, 5 Dec 2023 07:37:39 GMT, Markus KARG wrote: >> Brian Burkhalter has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8321053: instanceof -> == > > src/java.base/share/classes/java/io/ByteArrayInputStream.java line 213: > >> 211:

Re: RFR: 8321053: Use ByteArrayInputStream.buf directly when parameter of transferTo() is trusted [v4]

2023-12-04 Thread Markus KARG
On Mon, 4 Dec 2023 20:16:12 GMT, Brian Burkhalter wrote: >> Pass `ByteArrayInputStream.buf ` directly to the `OutputStream` parameter of >> `BAIS.transferTo` only if the target stream is in the `java.io` package. > > Brian Burkhalter has updated the pull request incrementally with one >

Re: RFR: 8321053: Use ByteArrayInputStream.buf directly when parameter of transferTo() is trusted [v4]

2023-12-04 Thread Brian Burkhalter
> Pass `ByteArrayInputStream.buf ` directly to the `OutputStream` parameter of > `BAIS.transferTo` only if the target stream is in the `java.io` package. Brian Burkhalter has updated the pull request incrementally with one additional commit since the last revision: 8321053: instanceof -> ==