On Thu, 11 Jun 2026 15:36:24 GMT, Viktor Klang <[email protected]> wrote:
>> David Simms has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains 2799 commits: >> >> - Merge remote-tracking branch 'valhalla/lworld' into 8317277 >> - 8386239: [lworld] Update jdk/java/util/Arrays/ArraysEqCmpTest.java >> >> Reviewed-by: liach >> - 8386242: [lworld] Simplify and clarify StrictProcessor >> >> Reviewed-by: dsimms >> - 8386140: [lworld] FieldReflector using wrong Class argument >> >> Reviewed-by: alanb, dsimms >> - 8386216: [lworld] Rollback meaningless diff in EventClassBuilder >> >> Reviewed-by: dsimms >> - Merge remote-tracking branch 'valhalla/lworld' into 8317277 >> - 8385170: [lworld] Serialization spec needs to allow abstract value >> classes like Number >> >> Reviewed-by: liach >> - 8385980: [lworld] Standardize pattern for preview value class generation >> >> Reviewed-by: liach >> - 8386086: [lworld] >> sun/jvmstat/monitor/MonitoredVm/MonitorVmStartTerminate.java should be >> removed from problem list >> >> Reviewed-by: liach, dcubed >> - Merge >> >> Merge jdk-28+1 >> - ... and 2789 more: https://git.openjdk.org/jdk/compare/92298786...26c8f2e4 > > src/java.base/share/classes/java/lang/Object.java line 523: > >> 521: * <div class="preview-comment"> >> 522: * If this object is a {@linkplain >> java.util.Objects#hasIdentity value object}, >> 523: * it does does not have a monitor, an {@code >> IllegalMonitorStateException} is thrown. > > Suggestion: > > * it does not have a monitor, an {@code > IllegalMonitorStateException} is thrown. As per PR body description: > [!NOTE] > The "*sub-review pull requests*" contain the same full set of code changes as > this "*master pull request*" to preserve the full implementation context; the > language compiler, JVM, and standard library changes are intertwined. The > separate pull requests exist only to subdivide the review and related > discussion by area. Please continue your comments here: https://github.com/openjdk/jdk/pull/31123 ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/31120#discussion_r3429038836
