On Tue, 7 Oct 2025 12:34:25 GMT, Johny Jose <[email protected]> wrote:

>> The permanent disabling of the Security Manager in JDK 24  allow setCCL to 
>> set the CCL to a custom class loader. However, it doesn't allow it to be set 
>> to a custom class loader and then "reset" back to the system class loader 
>> before executing further code in the task. The changes are made to relax the 
>> restriction in setCCL to reset back to system class loader
>
> Johny Jose has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Review changes

test/jdk/java/util/concurrent/forkjoin/ContextClassLoaderTest.java line 40:

> 38: import java.util.concurrent.ForkJoinPool;
> 39: import java.util.concurrent.TimeUnit;
> 40: import static org.junit.jupiter.api.Assertions.*;

The updated test looks okay.  Can you tidy up the imports as you've ended up 
with (harmless) duplicates which I assume you didn't intend.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27648#discussion_r2410535247

Reply via email to