<i18n dev> RFR: 8344039: Remove security manager dependency in java.time [v2]
Sean Mullan
mullan at openjdk.org
Wed Nov 13 14:31:41 UTC 2024
On Wed, 13 Nov 2024 14:12:34 GMT, Roger Riggs <rriggs at openjdk.org> wrote:
>> After [JDK-8338411](https://bugs.openjdk.org/browse/JDK-8338411), Serialization implementation dependencies on doPriviledged and AccessController are removed.
>> Some refactoring to cleanup the remaining code is expected.
>
> 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.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22042#discussion_r1840369340
More information about the i18n-dev
mailing list