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