On Wed, 11 Feb 2026 20:36:52 GMT, Alex Menkov <[email protected]> wrote:

> The fix updates the test to not synchronize on `Integer`, and use `Object` 
> instead

Changes requested by lmesnik (Reviewer).

test/jdk/com/sun/jdi/EATests.java line 2408:

> 2406:         printStack(bpe.thread());
> 2407:         @SuppressWarnings("unused")
> 2408:         ObjectReference o = getLocalRef(bpe.thread().frame(1), 
> Object.class.getName(), "l1");

The testcase tried to synchronize on the ValueBased class intentionally. So it 
can't be just updated to use non-value based class. 

I don't know how to implement the regression test for 
https://bugs.openjdk.org/browse/JDK-8324881 though.

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

PR Review: https://git.openjdk.org/jdk/pull/29679#pullrequestreview-3787785888
PR Review Comment: https://git.openjdk.org/jdk/pull/29679#discussion_r2795825702

Reply via email to