This bug has been created to address some of the valid review comments raised during review of JDK-8195799.
In JavaFX codebase -
1) com.sun.javafx.logging.PlatformLogger class was introduced while fixingJDK-8195799. Consider dropping com.sun.javafx.logging.PlatformLogger completely and directly use System.Logger instead.
2) Use System.Logger.Levels instead of com.sun.javafx.logging.PlatformLogger.Levels
Another suggestion is -
In com.sun.javafx.logging.PlatformLogger class, we don't need the loggers map.
getLogger() method can simply return - new PlatformLogger(System.getLogger(name));
In JavaFX codebase -
1) com.sun.javafx.logging.PlatformLogger class was introduced while fixing
2) Use System.Logger.Levels instead of com.sun.javafx.logging.PlatformLogger.Levels
Another suggestion is -
In com.sun.javafx.logging.PlatformLogger class, we don't need the loggers map.
getLogger() method can simply return - new PlatformLogger(System.getLogger(name));
- relates to
-
JDK-8195799 Use System logger instead of platform logger in javafx modules
- Resolved
-
JDK-8195974 Replace use of java.util.logging in javafx with System logger
- Resolved
-
JDK-8209036 systemTests should use PlatformLogger instead of java.util.logging
- Open