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