RFR: 8289572: InputStream wrapping with BufferedInputStream is redundant in HttpTimestamper
Jaikiran Pai
jpai at openjdk.org
Fri Jul 22 06:11:05 UTC 2022
On Fri, 1 Jul 2022 08:06:10 GMT, Сергей Цыпанов <duke at openjdk.org> wrote:
> In case we read all bytes from an `InputStream` we don't need wrapping with `BufferedInputStream` as the bytes are not written into internal buffer. With removal of redundant buffer we save 8 kB of allocated memory.
I see that Sean has approved this PR. I'll run some tests internally with this change and if it goes fine, then I'll go ahead and sponsor this today.
-------------
PR: https://git.openjdk.org/jdk/pull/9343
More information about the security-dev
mailing list