From 1dd515138307b4b165e975f1653f00ce79f0eda6 Mon Sep 17 00:00:00 2001
From: Ben Fry <fry@processing.org>
Date: Fri, 19 Jul 2013 14:46:28 -0700
Subject: [PATCH] more notes regarding #1955

---
 core/src/processing/core/PApplet.java | 30 +++++++++------------------
 core/todo.txt                         |  2 ++
 todo.txt                              |  2 --
 3 files changed, 12 insertions(+), 22 deletions(-)

diff --git a/core/src/processing/core/PApplet.java b/core/src/processing/core/PApplet.java
index c8de9b75b..4211a1c83 100755
--- a/core/src/processing/core/PApplet.java
+++ b/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);
diff --git a/core/todo.txt b/core/todo.txt
index f0307606c..a0f15a529 100644
--- a/core/todo.txt
+++ b/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 #)
diff --git a/todo.txt b/todo.txt
index c25fa95d8..49d968ace 100644
--- a/todo.txt
+++ b/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...