Re: RFR: 8343039: Remove jdk.internal.misc.InternalLock and usages from java.io [v9]

2024-11-14 Thread Chen Liang
On Thu, 14 Nov 2024 19:16:34 GMT, Brian Burkhalter wrote: >> Uses of `InternalLock` are removed and `synchronized` is reinstated. > > Brian Burkhalter has updated the pull request with a new target base due to a > merge or a rebase. The incremental webrev excludes the unrelated changes > brough

Re: RFR: 8343039: Remove jdk.internal.misc.InternalLock and usages from java.io [v9]

2024-11-14 Thread Brian Burkhalter
On Fri, 15 Nov 2024 01:20:43 GMT, Chen Liang wrote: > Looks good Thanks, @liach , for the second review. > src/java.base/share/classes/java/io/BufferedReader.java line 329: > >> 327: if (term != null) term[0] = false; >> 328: >> 329: bufferLoop: > > How do we usually

Re: RFR: 8343039: Remove jdk.internal.misc.InternalLock and usages from java.io [v9]

2024-11-14 Thread Alan Bateman
On Thu, 14 Nov 2024 19:16:34 GMT, Brian Burkhalter wrote: >> Uses of `InternalLock` are removed and `synchronized` is reinstated. > > Brian Burkhalter has updated the pull request with a new target base due to a > merge or a rebase. The incremental webrev excludes the unrelated changes > brough

Re: RFR: 8343039: Remove jdk.internal.misc.InternalLock and usages from java.io [v9]

2024-11-14 Thread Brian Burkhalter
> Uses of `InternalLock` are removed and `synchronized` is reinstated. Brian Burkhalter 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 nine additional