On Wed, 6 Mar 2024 14:14:56 GMT, Jaikiran Pai <j...@openjdk.org> wrote:
>> Archie Cobbs has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains six additional >> commits since the last revision: >> >> - Merge branch 'master' into JDK-7036144 >> - Merge branch 'master' into JDK-7036144 >> - Address third round of review comments. >> - Address second round of review comments. >> - Address review comments. >> - Fix bug in GZIPInputStream when underlying available() returns short. > > Hello Archie, I hope to finish off running the necessary analysis to see if > there is any obvious impact because of this change, in the coming days. Based > on the intial runs, the changes proposed in this PR look OK to me. > > In the meantime, given the nature of this change, I am marking this as > requiring a CSR. Would you be willing to come up with the CSR text for this > (https://wiki.openjdk.org/display/csr/Main)? Hi @jaikiran, > Would you be willing to come up with the CSR text for this Happy to - please see [JDK-8327489](https://bugs.openjdk.org/browse/JDK-8327489). ------------- PR Comment: https://git.openjdk.org/jdk/pull/17113#issuecomment-1981533001