RFR: 8236448: Remove unused and repair broken Android/Dalvik code

Kevin Rushforth kcr at openjdk.java.net
Fri Jan 3 23:19:16 UTC 2020


On Mon, 30 Dec 2019 11:05:55 GMT, Johan Vos <jvos at openjdk.org> wrote:

> This allows to build the JavaFX jars and native libraries for Android devices

I did a test build on Linux since there were changes to the shared Monocle classes. I didn't really look at the `android.gradle` changes or the new `android/nativeBridge.c` file, but I don't have any concerns. I did leave a couple minor formatting comments and one copyright year issue.

modules/javafx.graphics/src/main/java/com/sun/glass/ui/monocle/AndroidInputDeviceRegistry.java line 62:

> 61:         }
> 62:         Platform.runLater( () -> instance.gotTouchEvent(touchState));
> 63:     }

Minor: normally we wouldn't put a space between the initial `(` and the `()`

modules/javafx.graphics/src/main/java/com/sun/glass/ui/monocle/MonocleApplication.java line 229:

> 228:                                    ns.getWidth(), ns.getHeight(),
> 229:                                    1.f,1.f, ns.getScale(), ns.getScale());
> 230:         // Move the cursor to the middle of the screen

Minor: add a space after the first `,`

modules/javafx.graphics/src/main/java/com/sun/glass/ui/monocle/MonocleWindowManager.java line 175:

> 174:         Platform.runLater(new Runnable () {
> 175:             @Override
> 176:             public void run() {

Other than removing the one blank line above, this file is unchanged. Perhaps it could be reverted?

modules/javafx.graphics/src/main/native-glass/monocle/android/nativeBridge.h line 2:

> 1: /*
> 2:  * Copyright (c) 2019, Oracle and/or its affiliates. All rights reserved.
> 3:  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.

That should be `2012, 2019, Oracle...` so as not to lose the initial year (well actually `2012, 2020, Oracle...`  as of Jan 1, but that will get updated the next time I run my copyright script, which I plan to do right before RDP2, so either 2019 or 2020 for the last-modified year is fine).

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



PR: https://git.openjdk.java.net/jfx/pull/75


More information about the openjfx-dev mailing list