On Fri, 25 Jun 2021 14:54:45 GMT, Roger Riggs <rri...@openjdk.org> wrote:

>> Remove the unnecessary special case "OVERRIDE" in jdk.serialFilterFactory 
>> property.
>> Fix description in the example of a filter allowing platform classes.
>> Suppress some warnings about use of SecurityManager in tests.
>
> Roger Riggs has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Update logging of faults in jdk.serialFilterFactory to log only the 
> exception message
>   Simplify the logging.properties to only the needed settings

Marked as reviewed by bchristi (Reviewer).

test/jdk/java/io/Serializable/serialFilter/SerialFilterFunctionTest.java line 
49:

> 47:         // Enable logging
> 48:         System.setProperty("java.util.logging.config.file",
> 49:                 System.getProperty("test.src", ".") + 
> "/logging.properties");

Is `System.setProperty()` needed if it's already being set with -D ?

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

PR: https://git.openjdk.java.net/jdk17/pull/85

Reply via email to