RFR: 8344034: Remove security manager dependency in Serialization [v6]
Sean Mullan
mullan at openjdk.org
Thu Nov 14 14:01:58 UTC 2024
On Wed, 13 Nov 2024 15:35:42 GMT, Roger Riggs <rriggs at openjdk.org> wrote:
>> After [JDK-8338411](https://bugs.openjdk.org/browse/JDK-8338411), Serialization implementation dependencies on SecurityManager, doPrivildged, 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 unused imports
src/java.base/share/classes/java/io/ObjectOutputStream.java line 225:
> 223: * as true or false for extended information about exception's place
> 224: */
> 225: private static final boolean extendedDebugInfo = Boolean.getBoolean("sun.io.serialization.extendedDebugInfo");
Nit: bit of a long line, consider breaking into 2 lines.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22041#discussion_r1842268579
More information about the core-libs-dev
mailing list