Yes it could be converted, but wasn't necessary to remove the depenency,
so I filed a separate JBS issue to track this replacing the calls to
j.u.l: JDK-8195974 <https://bugs.openjdk.java.net/browse/JDK-8195974>.
-- Kevin
Tom Schindl wrote:
Minor question on this: I see the test stuff is using java.util.Logging
could this not be ported to PlatformLogger?
Tom
On 26.03.18 22:46, Kevin Rushforth wrote:
This looks fine to me now.
-- Kevin
Ajit Ghaisas wrote:
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 -
modules/javafx.base/src/main/java/com/sun/javafx/collections/SetListenerHelper.java
modules/javafx.graphics/src/main/java/javafx/scene/Scene.java
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.
Regards,
Ajit
-----Original Message-----
From: Kevin Rushforth Sent: Saturday, March 24, 2018 3:27 AM
To: Ajit Ghaisas
Cc: Mandy Chung; Daniel Fuchs; openjfx-dev@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
issue:
1. There is a typo in the Copyright year (201 rather than 2018) in the
following two files:
modules/javafx.base/src/main/java/com/sun/javafx/collections/SetListenerHelper.java
modules/javafx.graphics/src/main/java/javafx/scene/Scene.java
2. There are tab characters in the following file that need to be
changed to spaces:
modules/javafx.base/src/test/java/test/com/sun/javafx/binding/SelectBindingTest.java
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
