RFR: 8344039: Remove security manager dependency in java.time [v2]
Alan Bateman
alanb at openjdk.org
Wed Nov 13 14:41:57 UTC 2024
On Wed, 13 Nov 2024 14:13:59 GMT, Sean Mullan <mullan at openjdk.org> wrote:
>> Roger Riggs has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Remove unnecessary suppress warnings removal
>> ServiceLoader configuration exceptions (in a static initialization) are rethrown as Error.
>> Remove dependency on SecurityException thrown by ServiceLoader (as obsolete)
>
> src/java.base/share/classes/java/time/zone/ZoneRulesProvider.java line 1:
>
>> 1: /*
>
> Lines 171-172:
>
> if (ex.getCause() instanceof SecurityException) {
> continue; // ignore the security exception, try the next provider
> }
>
> You may be able to remove this code, but may be better to leave it as-is for now until the SL code has been checked for cleanup.
The SL changes are in JDK-8344011.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22042#discussion_r1840444817
More information about the core-libs-dev
mailing list