fix: patch for the main thread (awt | lwjgl3)

jme3-lwjgl3/src/main/java/com/jme3/system/lwjgl/LwjglCanvas.java

@@ -340,16 +340,14 @@ public void removeNotify() {
                 hasNativePeer.set(false);
                 reinitcontext.set(true);
 
-                 while (reinitcontext.get()) {
+                while (reinitcontext.get() && parallel.get()) {
                     try {
                         lock.wait();
                     } catch (InterruptedException ex) {
                         super.removeNotify();
                         return;
                     }
                 }
-
-                 reinitcontext.set(false);
             }
 
             // GL context is dead at this point
@@ -411,6 +409,13 @@ public Graphics getGraphics() {
     /** Notify if there is a change in canvas dimensions. */
     private final AtomicBoolean needResize = new AtomicBoolean(false);
 
+    /**
+     * Flag indicating whether a custom thread is used to separate GL rendering
+     * from the EDT; its value is false if the main thread is used via the
+     * {@code SwingUtilities.invokeLater() } function.
+     */
+    private final AtomicBoolean parallel = new AtomicBoolean(false);
+
     /**
      * Flag that uses the context to check if it is initialized or not, this prevents
      * it from being initialized multiple times and potentially breaking the JVM.
@@ -526,7 +531,6 @@ public void run() {
             if (needResize.getAndSet(false)) {
                 settings.setResolution(framebufferWidth, framebufferHeight);
                 listener.reshape(framebufferWidth, framebufferHeight, framebufferWidth, framebufferHeight);
-                listener.reshape(framebufferWidth, framebufferHeight);
             }
 
             synchronized (lock) {
@@ -588,9 +592,6 @@ public void run() {
                             } finally {
                                 canvas.unlock();
                             }
-
-                            // Sync the display on some systems.
-                            Toolkit.getDefaultToolkit().sync();
                         }
                     } catch (Throwable ex) {
                         listener.handleError("Error while swapping buffers", ex);
@@ -607,6 +608,17 @@ public void run() {
             if (needClose.get()) {
                 break;
             }
+
+            if (! parallel.get()) {
+                // Sync the display on some systems.
+                Toolkit.getDefaultToolkit().sync();
+                break;
+            }
+        }
+
+        if (!parallel.get() && !needClose.get()) {
+            SwingUtilities.invokeLater(() -> run());
+            return;
         }
 
         deinitInThread();
@@ -669,6 +681,12 @@ public void create(boolean waitFor) {
         if (this.contextFlag.get()) {
             return;
         }
+        /*
+         * Note that JME does not run on a thread parallel to the AWT EDT; 
+         * this applies only to macOS.
+         */
+        this.parallel.set(Platform.get() != Platform.MACOSX);
+
         // create context
         super.create(waitFor);
         this.contextFlag.set(true);
@@ -720,8 +738,8 @@ protected void createContext(AppSettings settings) {
 
         RENDER_CONFIGS.computeIfAbsent(settings.getRenderer(), (t) -> {
             return (data) -> {
-                data.majorVersion = 2;
-                data.minorVersion = 0;
+                data.majorVersion = 3;
+                data.minorVersion = 2;
             };
         }).accept(glData);
 
@@ -759,9 +777,14 @@ protected void createContext(AppSettings settings) {
         glData.forwardCompatible = false;
 
         allowSwapBuffers = settings.isSwapBuffers();
-
         canvas.createContext();
-        canvas.makeCurrent();
+
+        try {
+            canvas.lock();
+            canvas.makeCurrent();
+        } finally {
+            canvas.unlock();
+        }
 
         SwingUtilities.invokeLater(() -> {
             canvas.validate();