On Tue, 8 Sep 2026 12:32:31 GMT, Jorn Vernee <[email protected]> wrote:

>> test/jdk/java/foreign/TestMappedSegmentKeepAlive.java line 72:
>> 
>>> 70:             Thread t = Thread.ofPlatform()
>>> 71:                     // Provoke a WrongThreadException when acquiring 
>>> the session
>>> 72:                     // Make sure we properly release the session again
>> 
>> If this assertion fails, the exception is not propagated to the main thread, 
>> and the test succeeds. I know we are not testing WTE but would it make sense 
>> to either propagate the exception or just swallow WTE in the worker thread?
>
> Good point. I think since we have an expectation in this test that a WTE will 
> be thrown, we should check that. I'll propagate the exception to the main 
> thread.

I've added exception propagation to the test.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/31918#discussion_r3960555504

Reply via email to