JDK-8200236 : Simplify/Replace com.sun.javafx.logging.PlatformLogger class & its usage
  • Type: Bug
  • Component: javafx
  • Sub-Component: other
  • Affected Version: openjfx11
  • Priority: P4
  • Status: Open
  • Resolution: Unresolved
  • OS: generic
  • CPU: generic
  • Submitted: 2018-03-26
  • Updated: 2018-09-05
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
Other
tbdUnresolved
Related Reports
Relates :  
Relates :  
Relates :  
Description
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 fixing JDK-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));


Comments
The suggestions and discussion are in the mailing list thread for JDK-8195799: http://mail.openjdk.java.net/pipermail/openjfx-dev/2018-March/021610.html.
19-05-2018