[11] Review request : JDK-8195799 : Use System logger instead of platform logger in javafx modules

Ajit Ghaisas ajit.ghaisas at oracle.com
Mon Mar 26 11:36:15 UTC 2018

Thanks all for the review.

I have addressed the review comments in - http://cr.openjdk.java.net/~aghaisas/fx/8195799/webrev.1/

The changes are -
1. Addressed the (c) year inaccuracies in files -

2. Removed tabs from modules/javafx.base/src/test/java/test/com/sun/javafx/binding/SelectBindingTest.java
3. Removed unused methods from com.sun.javafx.logging.PlatformLogger class.

Also, I have created a new bug JDK-8200236 to address some of the valid suggestions from Mandy and Daniel.

Request you to review the new webrev.


-----Original Message-----
From: Kevin Rushforth 
Sent: Saturday, March 24, 2018 3:27 AM
To: Ajit Ghaisas
Cc: Mandy Chung; Daniel Fuchs; openjfx-dev at openjdk.java.net
Subject: Re: [11] Review request : JDK-8195799 : Use System logger instead of platform logger in javafx modules

The only additional comments I have are couple typos and a white-space

1. There is a typo in the Copyright year (201 rather than 2018) in the following two files:


2. There are tab characters in the following file that need to be changed to spaces:

     public static void setUpClass() {
 >>>        System.err.println("SelectBindingTest : log messages are 
expected from these tests.");

All my testing looks good. With this patch I am now able to run applications with OpenJDK 10 + a standalone FX SDK with no qualified exports on the command line (as long as it doesn't use Swing interop).

-- Kevin

Ajit Ghaisas wrote:
> Hi Kevin, Mandy and Daniel,
>     Please review the changeset that removes dependency on sun.util.logging package from JavaFX code.
>     Bug :  https://bugs.openjdk.java.net/browse/JDK-8195799
>     Fix :  http://cr.openjdk.java.net/~aghaisas/fx/8195799/webrev.0/
>     Request you to review.
> Regards,
> Ajit

More information about the openjfx-dev mailing list