y'all should handle nulls too

benfry · benfry · commit 09d6ee6cda2a · 2014-10-30T16:57:44.000-04:00
diff --git a/core/src/processing/data/Table.java b/core/src/processing/data/Table.java
@@ -1301,22 +1301,24 @@ void saveStringODS(OutputStream output, String text) throws IOException {
     // save us from having to create the entire document in memory again before
     // writing to the file. So while it's dorky, the outcome is still useful.
     StringBuilder sanitized = new StringBuilder();
-    char[] array = text.toCharArray();
-    for (char c : array) {
-      if (c == '&') {
-        sanitized.append("&amp;");
-      } else if (c == '\'') {
-        sanitized.append("&apos;");
-      } else if (c == '"') {
-        sanitized.append("&quot;");
-      } else if (c == '<') {
-        sanitized.append("&lt;");
-      } else if (c == '>') {
-        sanitized.append("&rt;");
-      } else if (c < 32 || c > 127) {
-        sanitized.append("&#" + ((int) c) + ";");
-      } else {
-        sanitized.append(c);
+    if (text != null) {
+      char[] array = text.toCharArray();
+      for (char c : array) {
+        if (c == '&') {
+          sanitized.append("&amp;");
+        } else if (c == '\'') {
+          sanitized.append("&apos;");
+        } else if (c == '"') {
+          sanitized.append("&quot;");
+        } else if (c == '<') {
+          sanitized.append("&lt;");
+        } else if (c == '>') {
+          sanitized.append("&rt;");
+        } else if (c < 32 || c > 127) {
+          sanitized.append("&#" + ((int) c) + ";");
+        } else {
+          sanitized.append(c);
+        }
       }
     }
 
