On Mon, 26 Apr 2021 09:33:03 GMT, Pavel Rappo <pra...@openjdk.org> wrote:

> I had been looking for an example of a "properties" file when spotted a typo 
> in `logging.properties`. I decided to proofread the file. That resulted in 
> finding a few other issues.

Otherwise, these changes look fine.

src/java.logging/share/conf/logging.properties line 5:

> 3: #
> 4: # You can use a different file by specifying a filename
> 5: # in the java.util.logging.config.file system property.

Unless the `java.util.logging.config` property contains more than one item of 
information, the original "with" is more appropriate here.

src/java.logging/share/conf/logging.properties line 46:

> 44: java.util.logging.FileHandler.formatter = java.util.logging.XMLFormatter
> 45: 
> 46: # Limit the messages that are printed on the console to INFO and above.

"Limit the messages ..." could be more direct. Consider "Only print messages to 
the console if they have level INFO or above."

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

Changes requested by mr (Lead).

PR: https://git.openjdk.java.net/jdk/pull/3688

Reply via email to