On Fri, 22 Nov 2024 11:21:34 GMT, Jaikiran Pai <j...@openjdk.org> wrote:

>> Can I please get a review of this change which removes the 
>> `getPermissions()` method from the `URLClassLoader`? This addresses 
>> https://bugs.openjdk.org/browse/JDK-8343150.
>> 
>> With the SecurityManager implementation removed through JEP 486, this method 
>> of the `URLClassLoader` is no longer relevant.
>> 
>> No new tests have been added. tier1 and tier2 testing is currently in 
>> progress.
>
> Jaikiran Pai has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Daniel's suggestion - minor javadoc update

LGTM

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

Marked as reviewed by dfuchs (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/22179#pullrequestreview-2454191552

Reply via email to