RFR: 8289572: InputStream wrapping with BufferedInputStream is redundant in HttpTimestamper

Jaikiran Pai jpai at openjdk.org
Tue Jul 5 06:20:25 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.

Looks fine to me, but please wait for someone from the security team to review this, before merging.

-------------

Marked as reviewed by jpai (Reviewer).

PR: https://git.openjdk.org/jdk/pull/9343



More information about the security-dev mailing list