From 1d8442d6b181ea0d1bf92687e48d68493565708d Mon Sep 17 00:00:00 2001
From: benfry <fry@processing.org>
Date: Sat, 5 Jun 2010 19:34:10 +0000
Subject: [PATCH] todo notes and some minor svg edits

---
 android/core/src/processing/core/PShapeSVG.java | 12 ++++++++++++
 android/todo.txt                                | 16 ++++++++++++++++
 2 files changed, 28 insertions(+)

diff --git a/android/core/src/processing/core/PShapeSVG.java b/android/core/src/processing/core/PShapeSVG.java
index 2f7eeaedb..84f1d651a 100644
--- a/android/core/src/processing/core/PShapeSVG.java
+++ b/android/core/src/processing/core/PShapeSVG.java
@@ -257,6 +257,15 @@ public class PShapeSVG extends PShape {
 
     element = properties;
     name = properties.getStringAttribute("id");
+    // @#$(* adobe illustrator
+    if (name != null) {
+      while (true) {
+        String[] m = PApplet.match(name, "_x(.*)_");
+        if (m == null) break;
+        char repair = (char) PApplet.unhex(m[1]);
+        name = name.replace(m[0], "" + repair);
+      }
+    }
 
     String displayStr = properties.getStringAttribute("display", "inline");
     visible = !displayStr.equals("none");
@@ -355,6 +364,9 @@ public class PShapeSVG extends PShape {
     } else if (name.equals("mask")) {
       PGraphics.showWarning("Masks are not supported.");
 
+    } else if (name.equals("sodipodi:namedview")) {
+      // these are always in Inkscape files, the warnings get tedious  
+
     } else {
       PGraphics.showWarning("Ignoring  <" + name + "> tag.");
     }
diff --git a/android/todo.txt b/android/todo.txt
index 95283807f..a3e10ebaa 100644
--- a/android/todo.txt
+++ b/android/todo.txt
@@ -1,6 +1,22 @@
 0185 android
 X fix two bugs with fonts created with specific charsets
+X fix for adobe illustrator-mangled svg id names with hex characters
 
+_ need to side-port changes from PShape/SVG over to Android
+
+_ image() problems (includes sketch)
+_   http://dev.processing.org/bugs/show_bug.cgi?id=1565
+
+_ move to Android 2.1 as the minimum requirement? 
+_   since 2.0.1 seems to be used on only a 0.46% of devices?
+_   http://developer.android.com/resources/dashboard/platform-versions.html
+
+_ add registered methods again
+_   need to figure out generic event queueing first
+_   may need a different subset of methods, and introduce new ones
+_     that will be usable on both android and desktop
+_ add line for export in libraries to say whether they're compatible
+_   even just 'android=' will be ok
 
 _ possibility of doing a compile (not run) using straight javac?
 _   this would be a faster way to check for errors