On Wed, 13 Nov 2024 21:39:54 GMT, Brian Burkhalter <b...@openjdk.org> wrote:

>> Uses of `InternalLock` are removed and `synchronized` is reinstated.
>
> Brian Burkhalter has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   8343039: Remove failing getDeclaredField call from test 
> java/lang/ProcessBuilder/Basic.java

I've changed the PR to require at least two reviewers as I think the changes 
could benefit from a second reviewer.

src/java.base/share/classes/java/io/BufferedOutputStream.java line 89:

> 87: 
> 88:         if (getClass() == BufferedOutputStream.class) {
> 89:             // resizable if initialSize < maxSize

I think the comment is confusing, can it just say resizable when not 
sub-classed as it did previously?

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

PR Comment: https://git.openjdk.org/jdk/pull/22048#issuecomment-2476163286
PR Review Comment: https://git.openjdk.org/jdk/pull/22048#discussion_r1842102399

Reply via email to