From 5a91bbade028d99bbe08485c5bd31b5f2566c3d2 Mon Sep 17 00:00:00 2001 From: benfry Date: Sun, 2 Sep 2012 23:47:50 +0000 Subject: [PATCH] remove clear() after alll --- core/src/processing/core/PGraphics.java | 5 --- core/src/processing/core/PGraphicsJava2D.java | 13 ++++--- core/todo.txt | 32 ++++++++-------- todo.txt | 37 +++++++++---------- 4 files changed, 40 insertions(+), 47 deletions(-) diff --git a/core/src/processing/core/PGraphics.java b/core/src/processing/core/PGraphics.java index 324d37e57..e87f355bc 100644 --- a/core/src/processing/core/PGraphics.java +++ b/core/src/processing/core/PGraphics.java @@ -6637,11 +6637,6 @@ public class PGraphics extends PImage implements PConstants { // BACKGROUND - public void clear() { - showMethodWarning("clear"); - } - - /** * ( begin auto-generated from background.xml ) * diff --git a/core/src/processing/core/PGraphicsJava2D.java b/core/src/processing/core/PGraphicsJava2D.java index 3cd339176..d20f384d8 100644 --- a/core/src/processing/core/PGraphicsJava2D.java +++ b/core/src/processing/core/PGraphicsJava2D.java @@ -160,6 +160,13 @@ public class PGraphicsJava2D extends PGraphics /*PGraphics2D*/ { g2 = (Graphics2D) image.getGraphics(); } + // Avoid badness when drawing shorter strokes. + // http://code.google.com/p/processing/issues/detail?id=1068 + // With very small stroke increments, this was actually faster by 2x, + // and for larger increments it was only 3% slower. + g2.setRenderingHint(RenderingHints.KEY_STROKE_CONTROL, + RenderingHints.VALUE_STROKE_PURE); + // can't un-set this because this may be only a resize // http://dev.processing.org/bugs/show_bug.cgi?id=463 //defaultsInited = false; @@ -1818,12 +1825,6 @@ public class PGraphicsJava2D extends PGraphics /*PGraphics2D*/ { // BACKGROUND - @Override - public void clear() { - clearPixels(0); - } - - int[] clearPixels; protected void clearPixels(int color) { diff --git a/core/todo.txt b/core/todo.txt index 37f0d3edb..f4082949d 100644 --- a/core/todo.txt +++ b/core/todo.txt @@ -4,32 +4,30 @@ X http://code.google.com/p/processing/issues/detail?id=1073 X maybe getResource() fails because spaces are not encoded? X PVector limit() is inefficient X http://code.google.com/p/processing/issues/detail?id=1122 - +X memory leak when many createGraphics(..., JAVA2D) calls are used +X http://code.google.com/p/processing/issues/detail?id=507 +X need to add clear() method so clear out ARGB surfaces +C clear() ok to add +X decided clear(r, g, b, a) problematic since it's an exception to color model +X and that people should use background() anyway +o or background(r, g, b, a) would be the thing +X clear() with a color doesn't make (verbal) sense +X provide a way to clear the PGraphics with plain alpha +X removed in the end... background() can already do it X work on making API more generic and consistent X PFont.getFont() changed to PFont.getNative() (returns an Object) X PFont.getTypeface() changed to PFont.getNative() +X PImage.getBitmap() and getImage() changed to PImage.getNative() +X use getNative() to return the native object for +X PImage (BufferedImage and Bitmap), PFont (Typeface or awt.Font) earlier X add breakShape to the public API -> begin/endContour x or handle differently in general.. nested beginPath() calls? - _ beginning slash in getChild() threw an NPE _ do we need a trim() method for XML? or enableWhiteSpace()? or? -_ memory leak when many createGraphics(..., JAVA2D) calls are used -_ http://code.google.com/p/processing/issues/detail?id=507 -_ need to add clear() method so clear out ARGB surfaces -C clear() ok to add -X decided clear(r, g, b, a) problematic since it's an exception to color model -X and that people should use background() anyway -o or background(r, g, b, a) would be the thing -_ clear() with a color doesn't make (verbal) sense -_ provide a way to clear the PGraphics with plain alpha - -_ use getNative() to return the native object for -_ PImage (BufferedImage and Bitmap), PFont (Typeface or awt.Font) - api to be fixed/removed _ remove PImage.delete() and friends from PImage, Movie, etc. _ delete()/dispose() being used in the movie @@ -43,7 +41,9 @@ _ make PShapeOBJ the 3D loader o separate PShape2/3D from OpenGL o incorporate SVG loader into the 2D class -_ loadPixels() implementation +_ loadShape() needs to live in PApplet + +_ loadPixels() implementation needs to be in PApplet, not PGraphics _ Jagged / Glitchy JAVA2D shape strokes in Java 1.6 _ g2.setRenderingHint(RenderingHints.KEY_STROKE_CONTROL, diff --git a/todo.txt b/todo.txt index d8a76756d..5f81f42b3 100644 --- a/todo.txt +++ b/todo.txt @@ -12,6 +12,11 @@ _ http://code.google.com/p/processing/issues/detail?id=1056 _ remove preferences that refer to applets _ remove applet folder from the Java stuff +_ 'recent' menu doesn't respect examples folder of other p5 versions +_ 'recent' menu paths can get enormous + +. . . + _ try to use appbundler to create a version that includes a JRE (JDK) _ http://java.net/downloads/appbundler/appbundler.html _ http://docs.oracle.com/javase/7/docs/technotes/guides/jweb/packagingAppsForMac.html @@ -30,38 +35,30 @@ an operation that would require enabling a specific entitlement, like connecting to a network, printing, interacting with a usb or bluetooth device, etc.. -. . . - I encountered this problem too, I forgot to add it to my guide.... If you sign all the files in the bundle it won't work as codesign doesn't follow the symlinks. - First sign your bundle: -codesign --verbose -f -s "$SIGNATURE_APP" --entitlements $ENTITLEMENTS -$YOUR_APP.app - +codesign --verbose -f -s "$SIGNATURE_APP" \ + --entitlements $ENTITLEMENTS \ + $YOUR_APP.app Then sign all the libraries: -find $YOUR_APP/Contents/ -type f \( -name "*.jar" -or -name "*.dylib" -\) -exec codesign --verbose -f -s "$SIGNATURE_APP" --entitlements -$ENTITLEMENTS {} \; - +find $YOUR_APP/Contents/ -type f \ + \( -name "*.jar" -or -name "*.dylib"\) \ + -exec codesign --verbose -f -s "$SIGNATURE_APP" \ + --entitlements $ENTITLEMENTS {} \; Finally you can create the package: -productbuild --component YOUR_APP.app /Applications --sign -"$SIGNATURE_INST" YOUR_APP.pkg - +productbuild --component YOUR_APP.app /Applications \ + --sign "$SIGNATURE_INST" YOUR_APP.pkg You can test if the package work with this command: - sudo installer -store -pkg $YOUR_APP.pkg -target / - You can also verify all libraries have been signed -find YOUR_APP/Contents/ -type f \( -name "*.jar" -or -name "*.dylib" -\) -exec codesign --verbose --verify {} \; +find YOUR_APP/Contents/ -type f \ + \( -name "*.jar" -or -name "*.dylib"\) \ + -exec codesign --verbose --verify {} \; . . . -_ 'recent' menu doesn't respect examples folder of other p5 versions -_ 'recent' menu paths can get enormous - 2.0 FINAL / command line _ kill external editor (for all its quirks) in favor of command line