more notes regarding #1955

core/src/processing/core/PApplet.java

@@ -10379,33 +10379,23 @@ public class PApplet extends Applet
       displayDevice = environment.getDefaultScreenDevice();
     }
 
-    // Using a JFrame fixes a Windows problem with Present mode.
-    // This might be our error, but usually this is the sort of crap
-    // we usually get from OS X.
+    // Using a JFrame fixes a Windows problem with Present mode. This might
+    // be our error, but usually this is the sort of crap we usually get from
+    // OS X. It's time for a turnaround: Redmond is thinking different too!
+    // https://github.com/processing/processing/issues/1955
     Frame frame = new JFrame(displayDevice.getDefaultConfiguration());
-    frame.setBackground(new Color(0xCC, 0xCC, 0xCC)); // default Processing gray
-//    JFrame frame = new JFrame(displayDevice.getDefaultConfiguration());
-      /*
-      Frame frame = null;
-      if (displayDevice != null) {
-        frame = new Frame(displayDevice.getDefaultConfiguration());
-      } else {
-        frame = new Frame();
-      }
-      */
-      //Dimension screen = Toolkit.getDefaultToolkit().getScreenSize();
-
-    // remove the grow box by default
-    // users who want it back can call frame.setResizable(true)
-//    frame.setResizable(false);
-    // moved later (issue #467)
+    // Default Processing gray, which will be replaced below if another
+    // color is specified on the command line (i.e. in the prefs).
+    frame.setBackground(new Color(0xCC, 0xCC, 0xCC));
+    // Cannot call setResizable(false) until later due to OS X (issue #467)
 
     final PApplet applet;
     if (constructedApplet != null) {
       applet = constructedApplet;
     } else {
       try {
-        Class<?> c = Thread.currentThread().getContextClassLoader().loadClass(name);
+        Class<?> c =
+          Thread.currentThread().getContextClassLoader().loadClass(name);
         applet = (PApplet) c.newInstance();
       } catch (Exception e) {
         throw new RuntimeException(e);

core/todo.txt

@@ -11,6 +11,8 @@ X   https://github.com/processing/processing/issues/1959
 X   turns out this is an apple.awt tweak for the exported Info.plist
 X getSubset() broken in IntList, StringList, and missing from FloatList
 X   https://github.com/processing/processing/issues/1979
+X Present Mode trouble on Windows 7 (64-bit) completely broken
+X   https://github.com/processing/processing/issues/1955
 
 
 _ retain blendMode() between frames (get bug #)

todo.txt

@@ -43,8 +43,6 @@ _ init() not called on tools until later
 _   https://github.com/processing/processing/issues/1859
 
 high
-_ Present Mode trouble on Windows 7 (64-bit) completely broken
-_   https://github.com/processing/processing/issues/1955
 _ move old Google Code SVN back to processing.org
 _   then cull out the old branches/tags from the Github repo
 _ UnsatisfiedLinkError causes huge message...