On Fri, 29 May 2026 10:39:49 GMT, Jaikiran Pai <[email protected]> wrote:
>>> How do you plan to solve point "3. For JDK 27, introduce a system >>> property..."? Do you want to do this with the initial fix for >>> [JDK-7036144](https://bugs.openjdk.org/browse/JDK-7036144), by wrapping the >>> input stream and overriding available() or in a completely different way? >> >> The plan is to reintroduce the change that was done for JDK-7036144 and >> enable that new behaviour only when the new system property is enabled. When >> that system property isn't set (or disabled), which would be the default >> case, then the implementation will continue with the long standing behaviour >> of using `InputStream.available()`. > >> @jaikiran, is my understanding correct that by "reintroduce the change that >> was done for JDK-7036144" you implicitly also mean to reintroduce the fix >> [JDK-8374644](https://bugs.openjdk.org/browse/JDK-8374644) for the >> performance regression > > Correct. > Expand the test coverage. It's always risky to change something in code from > JDK 1.0/1.1 so having tests that capture the current behavior (even if > unspecified) is important before making any change. A new test has been proposed https://bugs.openjdk.org/browse/JDK-8385891 to help increase the coverage. Over time, more will be added. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/30925#discussion_r3348948353
