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