From 456b6b01b51d3edded288718a18c5056b7ca2381 Mon Sep 17 00:00:00 2001
From: benfry <fry@processing.org>
Date: Sat, 8 Dec 2012 02:33:52 +0000
Subject: [PATCH] looking into Windows 8 MovieMaker issue

---
 .../src/processing/app/tools/MovieMaker.java  | 25 ++++++++++++-------
 core/todo.txt                                 |  2 ++
 todo.txt                                      |  2 ++
 3 files changed, 20 insertions(+), 9 deletions(-)

diff --git a/build/shared/tools/MovieMaker/src/processing/app/tools/MovieMaker.java b/build/shared/tools/MovieMaker/src/processing/app/tools/MovieMaker.java
index 75bef51f1..4f56cfe1f 100644
--- a/build/shared/tools/MovieMaker/src/processing/app/tools/MovieMaker.java
+++ b/build/shared/tools/MovieMaker/src/processing/app/tools/MovieMaker.java
@@ -2,6 +2,7 @@ package processing.app.tools;
 
 import java.io.IOException;
 
+import processing.app.Base;
 import processing.app.Editor;
 
 
@@ -20,15 +21,21 @@ public class MovieMaker implements Tool {
 
 
   public void run() {
-    String classPath =
-      getClass().getProtectionDomain().getCodeSource().getLocation().getPath();
-    System.out.println("cp is " + classPath);
-    try {
-      Runtime.getRuntime().exec(new String[] {
-        "java", "-cp", classPath, "processing.app.tools.MovieMakerFrame"
-      });
-    } catch (IOException e) {
-      e.printStackTrace();
+    if (Base.isMacOS()) {
+      // For OS X, run out of process, so that Quaqua doesn't hose the layout.
+      // http://code.google.com/p/processing/issues/detail?id=836
+      String classPath =
+          getClass().getProtectionDomain().getCodeSource().getLocation().getPath();
+      try {
+        Runtime.getRuntime().exec(new String[] {
+          "java", "-cp", classPath, "processing.app.tools.MovieMakerFrame"
+        });
+      } catch (IOException e) {
+        e.printStackTrace();
+      }
+    } else {
+      System.out.println("null frame, yeah baby");
+      MovieMakerFrame.main(null);
     }
   }
 
diff --git a/core/todo.txt b/core/todo.txt
index ce0b2e6f0..d229096b1 100644
--- a/core/todo.txt
+++ b/core/todo.txt
@@ -1,5 +1,7 @@
 0216 core
 
+_ Add clear() to replace background(0, 0, 0, 0)
+_   http://code.google.com/p/processing/issues/detail?id=1446
 
 _ implement mousePressed(Event) etc
 _   better to do this instead of bringing back the magic event
diff --git a/todo.txt b/todo.txt
index a4013b187..24e603e95 100644
--- a/todo.txt
+++ b/todo.txt
@@ -1,5 +1,7 @@
 0216 pde
 
+_ on Windows, the Help menu seems to start with a separator
+
 
 https://processing-js.lighthouseapp.com/