rewrite lwjgl3 backend to use SDL

Author: riccardobl

.gitignore

@@ -45,4 +45,4 @@ javadoc_deploy.pub
 !.vscode/settings.json
 !.vscode/JME_style.xml
 !.vscode/extensions.json
-joysticks-*.txt
+joysticks-*.txt

common.gradle

@@ -34,6 +34,9 @@ repositories {
     flatDir {
         dirs rootProject.file('lib')
     }
+    maven {
+        url "https://maven.rblb.it/riccardobl/angle-natives"
+    }
 }
 
 dependencies {

gradle/libs.versions.toml

@@ -33,6 +33,8 @@ lwjgl3-openal   = { module = "org.lwjgl:lwjgl-openal",   version.ref = "lwjgl3"
 lwjgl3-opencl   = { module = "org.lwjgl:lwjgl-opencl",   version.ref = "lwjgl3" }
 lwjgl3-opengl   = { module = "org.lwjgl:lwjgl-opengl",   version.ref = "lwjgl3" }
 lwjgl3-sdl      = { module = "org.lwjgl:lwjgl-sdl",      version.ref = "lwjgl3" }
+lwjgl3-opengles = { module = "org.lwjgl:lwjgl-opengles", version.ref = "lwjgl3" }
+lwjgl3-egl   = { module = "org.lwjgl:lwjgl-egl", version.ref = "lwjgl3" }
 
 mokito-core = "org.mockito:mockito-core:3.12.4"
 

jme3-core/src/main/java/com/jme3/renderer/opengl/GLES_30.java

@@ -42,10 +42,14 @@ public interface GLES_30 extends GL {
 
     public static final int GL_RGB10_A2 = 0x8059;
     public static final int GL_UNSIGNED_INT_2_10_10_10_REV = 0x8368;
- 
+    public static final int GL_NUM_EXTENSIONS = 0x821D;
+
     public void glBindVertexArray(int array);
 
     public void glDeleteVertexArrays(IntBuffer arrays);
 
     public void glGenVertexArrays(IntBuffer arrays);
+
+    public String glGetString(final int name, final int index);
+
 }

jme3-core/src/main/java/com/jme3/renderer/opengl/GLRenderer.java

@@ -186,7 +186,15 @@ private HashSet<String> loadExtensions() {
             gl3.glGetInteger(GL3.GL_NUM_EXTENSIONS, intBuf16);
             int extensionCount = intBuf16.get(0);
             for (int i = 0; i < extensionCount; i++) {
-                String extension = gl3.glGetString(GL.GL_EXTENSIONS, i);
+                String extension = gl3.glGetString(GL3.GL_EXTENSIONS, i);
+                extensionSet.add(extension);
+            }
+        } else if (caps.contains(Caps.OpenGLES30)) {
+            GLES_30 gles = (GLES_30) gl;
+            gles.glGetInteger(GLES_30.GL_NUM_EXTENSIONS, intBuf16);
+            int extensionCount = intBuf16.get(0);
+            for (int i = 0; i < extensionCount; i++) {
+                String extension = gles.glGetString(GLES_30.GL_EXTENSIONS, i);
                 extensionSet.add(extension);
             }
         } else {

jme3-core/src/main/java/com/jme3/system/AppSettings.java

@@ -219,6 +219,8 @@ public final class AppSettings extends HashMap<String, Object> {
      */
     public static final String LWJGL_OPENAL = "LWJGL";
 
+    public static final String ANGLE_GLES3 = "ANGLE_GLES3";
+
     /**
      * Use the Android MediaPlayer / SoundPool based renderer for Android audio capabilities.
      * <p>
@@ -296,18 +298,18 @@ public final class AppSettings extends HashMap<String, Object> {
     static {
         defaults.put("Display", 0);
         defaults.put("CenterWindow", true);
-        defaults.put("Width", 640);
-        defaults.put("Height", 480);
+        defaults.put("Width", 1440);
+        defaults.put("Height", 900);
         defaults.put("WindowWidth", Integer.MIN_VALUE);
         defaults.put("WindowHeight", Integer.MIN_VALUE);
         defaults.put("BitsPerPixel", 24);
-        defaults.put("Frequency", 60);
+        defaults.put("Frequency", 0);
         defaults.put("DepthBits", 24);
         defaults.put("StencilBits", 0);
         defaults.put("Samples", 0);
         defaults.put("Fullscreen", false);
         defaults.put("Title", JmeVersion.FULL_NAME);
-        defaults.put("Renderer", LWJGL_OPENGL32);
+        defaults.put("Renderer", ANGLE_GLES3);
         defaults.put("AudioRenderer", LWJGL_OPENAL);
         defaults.put("DisableJoysticks", true);
         defaults.put("UseInput", true);
@@ -317,7 +319,7 @@ public final class AppSettings extends HashMap<String, Object> {
         defaults.put("MinHeight", 0);
         defaults.put("MinWidth", 0);
         defaults.put("GammaCorrection", true);
-        defaults.put("Resizable", false);
+        defaults.put("Resizable", true);
         defaults.put("SwapBuffers", true);
         defaults.put("OpenCL", false);
         defaults.put("OpenCLPlatformChooser", DefaultPlatformChooser.class.getName());

jme3-core/src/plugins/java/com/jme3/texture/plugins/TGALoader.java

@@ -72,6 +72,14 @@ public final class TGALoader implements AssetLoader {
     // 11 - run-length encoded, black and white image
     public static final int TYPE_BLACKANDWHITE_RLE = 11;
 
+    private static void convertBGRtoRGB(byte[] data, int dl){
+        for (int i = 0; i < data.length; i += dl) {
+            byte tmp = data[i];
+            data[i] = data[i + 2];
+            data[i + 2] = tmp;
+        }
+    }
+
     @Override
     public Object load(AssetInfo info) throws IOException {
         if (!(info.getKey() instanceof TextureKey)) {
@@ -261,7 +269,8 @@ public static Image load(InputStream in, boolean flip) throws IOException {
 //                        rawData[rawDataIndex++] = blue;
 //                    }
                 }
-                format = Format.BGR8;
+                convertBGRtoRGB(rawData, dl);
+                format = Format.RGB8;
             } else if (pixelDepth == 32) {
                 for (int i = 0; i <= (height - 1); i++) {
                     if (!flip) {

jme3-desktop/src/main/java/com/jme3/system/JmeDesktopSystem.java

@@ -216,7 +216,7 @@ public JmeContext newContext(AppSettings settings, Type contextType) {
                 || contextType == JmeContext.Type.Headless) {
             ctx = new NullContext();
             ctx.setSettings(settings);
-        } else if (settings.getRenderer().startsWith("LWJGL")) {
+        } else if (settings.getRenderer().startsWith("LWJGL") || settings.getRenderer().startsWith("ANGLE")) {
             ctx = newContextLwjgl(settings, contextType);
             ctx.setSettings(settings);
         } else if (settings.getRenderer().startsWith("JOGL")) {

jme3-desktop/src/main/java/com/jme3/system/NativeLibraryLoader.java

@@ -44,6 +44,7 @@
 import java.util.logging.Level;
 import java.util.logging.Logger;
 
+import com.jme3.system.NativeLibraries.LibraryInfo;
 import com.jme3.util.res.Resources;
 
 /**
@@ -94,6 +95,16 @@ public static void registerNativeLibrary(NativeLibrary library) {
         nativeLibraryMap.put(library.getKey(), library);
     }
 
+    /**
+     * Register a new native library.
+     *
+     * This simply registers a known library, the actual extraction and loading is performed by calling
+     * {@link #loadNativeLibrary(java.lang.String, boolean) }.
+     */
+    public static void registerNativeLibrary(LibraryInfo library) {
+        library.getNativeVariants().forEach(NativeLibraryLoader::registerNativeLibrary);
+    }
+
     /**
      * Register a new native library.
      * 
@@ -432,11 +443,15 @@ public static void extractNativeLibrary(Platform platform, String name, File tar
     /**
      * First extracts the native library and then loads it.
      * 
-     * @param name The name of the library to load.
-     * @param isRequired If true and the library fails to load, throw exception. If
-     * false, do nothing if it fails to load.
+     * @param name
+     *            The name of the library to load.
+     * @param isRequired
+     *            If true and the library fails to load, throw exception. If false, do nothing if it fails to
+     *            load.
+     * 
+     * @return The absolute path of the loaded library.
      */
-    public static void loadNativeLibrary(String name, boolean isRequired) {
+    public static String loadNativeLibrary(String name, boolean isRequired) {
         if (JmeSystem.isLowPermissions()) {
             throw new UnsupportedOperationException("JVM is running under "
                     + "reduced permissions. Cannot load native libraries.");
@@ -457,15 +472,15 @@ public static void loadNativeLibrary(String name, boolean isRequired) {
                                     " is not available for your OS: {1}",
                             new Object[]{name, platform});
                 }
-                return;
+                return null;
             }
         }
 
         final String pathInJar = library.getPathInNativesJar();
 
         if (pathInJar == null) {
             // This platform does not require the native library to be loaded.
-            return;
+            return null;
         }
 
         URL url = Resources.getResource(pathInJar);
@@ -480,7 +495,7 @@ public static void loadNativeLibrary(String name, boolean isRequired) {
                                 " was not found in the classpath via ''{1}''.",
                         new Object[]{library.getName(), pathInJar});
             }
-            return;
+            return null;
         }
 
         // The library has been found and is ready to be extracted.
@@ -530,6 +545,8 @@ public static void loadNativeLibrary(String name, boolean isRequired) {
             if (logger.isLoggable(Level.FINE)) {
                 logger.log(Level.FINE, "Loaded native library {0}.", library.getName());
             }
+
+            return targetFile.getAbsolutePath();
         } catch (IOException ex) {
             /*if (ex.getMessage().contains("used by another process")) {
                 return;

jme3-desktop/src/main/java/com/jme3/texture/plugins/AWTLoader.java

@@ -101,6 +101,27 @@ private void flipImage(short[] img, int width, int height, int bpp){
         }
     }
 
+    private static void convertBGRtoRGB(byte[] data){
+        for (int i = 0; i < data.length; i += 3) {
+            byte tmp = data[i];
+            data[i] = data[i + 2];
+            data[i + 2] = tmp;
+        }
+    }
+
+    private static void convertABGRtoRGBA(byte[] data){
+        for (int i = 0; i < data.length; i += 4) {
+            byte a = data[i];
+            byte b = data[i + 1];
+            byte g = data[i + 2];
+            byte r = data[i + 3];
+            data[i]     = r;
+            data[i + 1] = g;
+            data[i + 2] = b;
+            data[i + 3] = a;
+        }
+    }
+
     public Image load(BufferedImage img, boolean flipY){
         int width = img.getWidth();
         int height = img.getHeight();
@@ -110,18 +131,22 @@ public Image load(BufferedImage img, boolean flipY){
                byte[] dataBuf1 = (byte[]) extractImageData(img);
                if (flipY)
                    flipImage(dataBuf1, width, height, 32);
+
+               convertABGRtoRGBA(dataBuf1);
 
                ByteBuffer data1 = BufferUtils.createByteBuffer(img.getWidth()*img.getHeight()*4);
                data1.put(dataBuf1);
-               return new Image(Format.ABGR8, width, height, data1, null, com.jme3.texture.image.ColorSpace.sRGB);
+               return new Image(Format.RGBA8, width, height, data1, null, com.jme3.texture.image.ColorSpace.sRGB);
             case BufferedImage.TYPE_3BYTE_BGR: // most common in JPEG images
                byte[] dataBuf2 = (byte[]) extractImageData(img);
                if (flipY)
                    flipImage(dataBuf2, width, height, 24);
+
+               convertBGRtoRGB(dataBuf2);
 
                ByteBuffer data2 = BufferUtils.createByteBuffer(img.getWidth()*img.getHeight()*3);
                data2.put(dataBuf2);
-               return new Image(Format.BGR8, width, height, data2, null, com.jme3.texture.image.ColorSpace.sRGB);
+               return new Image(Format.RGB8, width, height, data2, null, com.jme3.texture.image.ColorSpace.sRGB);
             case BufferedImage.TYPE_BYTE_GRAY: // grayscale fonts
                 byte[] dataBuf3 = (byte[]) extractImageData(img);
                 if (flipY)