fixed bug

Author: duskvirkus

core/examples/src/main/java/Issue1003.java

@@ -0,0 +1,34 @@
+import processing.core.PApplet;
+import processing.awt.PSurfaceAWT;
+
+// Reproduction of issue #1003
+// Resizeable causes crash on linux mint
+public class Issue1003 extends PApplet {
+
+    public void settings(){
+        size(200, 200);
+    }
+
+    public void setup() {
+        ((PSurfaceAWT) surface).setTitle("Hello world!");
+        ((PSurfaceAWT) surface).setResizable(true);
+//        surface.setLocation(100, 100);
+    }
+
+    public void draw(){
+//        ((PSurfaceAWT) surface).setResizable(true);
+
+        background(frameCount % 255);
+        line(0, 0, width, height);
+        line(width, 0, 0, height);
+    }
+    public static void main(String[] passedArgs) {
+        String[] appletArgs = new String[]{ Issue1003.class.getName()};
+        if (passedArgs != null) {
+            PApplet.main(concat(appletArgs, passedArgs));
+        } else {
+            PApplet.main(appletArgs);
+        }
+
+    }
+}

core/src/processing/awt/PSurfaceAWT.java

@@ -49,14 +49,15 @@ public class PSurfaceAWT extends PSurfaceNone {
   GraphicsDevice displayDevice;
 
   // used for canvas to determine whether resizable or not
-//  boolean resizable;  // default is false
+  boolean resizable;  // default is false
 
   // Internally, we know it's always a JFrame (not just a Frame)
 //  JFrame frame;
   // Trying Frame again with a11 to see if this avoids some Swing nastiness.
   // In the past, AWT Frames caused some problems on Windows and Linux,
   // but those may not be a problem for our reworked PSurfaceAWT class.
   JFrame frame;
+  boolean frameSetupComplete;
 
   // Note that x and y may not be zero, depending on the display configuration
   Rectangle screenRect;
@@ -85,6 +86,8 @@ public PSurfaceAWT(PGraphics graphics) {
     //this.graphics = graphics;
     super(graphics);
 
+    this.resizable = false; // Default is false, can be changed with surface.setResizable()
+
     /*
     if (checkRetina()) {
 //      System.out.println("retina in use");
@@ -196,8 +199,7 @@ public Dimension getMinimumSize() {
 
     @Override
     public Dimension getMaximumSize() {
-      //return resizable ? super.getMaximumSize() : getPreferredSize();
-      return frame.isResizable() ? super.getMaximumSize() : getPreferredSize();
+      return resizable ? super.getMaximumSize() : getPreferredSize();
     }
 
 
@@ -442,7 +444,12 @@ public void initFrame(final PApplet sketch) {/*, int backgroundColor,
 
     // disabling resize has to happen after pack() to avoid apparent Apple bug
     // https://github.com/processing/processing/issues/506
-    frame.setResizable(false);
+    // Must use this.resizable to avoid bug where value is set before initFrame is finished
+    // https://github.com/processing/processing4/issues/1003
+    System.out.println("Resizable in initFrame: " + this.resizable);
+//    setResizable(this.resizable);
+//    frame.setResizable(true);
+//    frame.setResizable(this.resizable);
 
     frame.addWindowListener(new WindowAdapter() {
       @Override
@@ -484,11 +491,18 @@ public void setTitle(String title) {
   /** Set true if we want to resize things (default is not resizable) */
   @Override
   public void setResizable(boolean resizable) {
-    //this.resizable = resizable;  // really only used for canvas
+    this.resizable = resizable;  // we need to store this so if frame init is not complete then we know the value
 
+    System.out.println("Frame in setResizable:" + frame);
     if (frame != null) {
-      frame.setResizable(resizable);
+      frame.setResizable(this.resizable);
+      System.out.println("PSurfaceAWT.frame.resizable set to:" + frame.isResizable());
+//      frame.isValid();
+      frame.validate();
+      System.out.println("PSurfaceAWT.frame.isValid():" + frame.isValid());
     }
+
+    System.out.println("PSurfaceAWT.resizable set to:" + this.resizable);
   }
 
 

core/src/processing/core/PApplet.java

@@ -10145,8 +10145,15 @@ static public void runSketch(final String[] args,
     }
     */
 
+
+
     sketch.showSurface();
+
+    surface.setResizable(false);
+
+
     sketch.startSurface();
+
   }