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 did another pass over this and don't see any issues except the confusing comment in the BOS constructor. Overall good cleanup as InternalLock was a temporary crutch to avoid pinning in the java.io area for common usages. ------------- Marked as reviewed by alanb (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/22048#pullrequestreview-2435909813
