Changeset 9828 in josm for trunk/src/org/openstreetmap/josm/io

Timestamp:

2016-02-19T02:06:48+01:00 (10 years ago)

Author:

Don-vip

Message:

findbugs, javadoc, unit tests

File:

• trunk/src/org/openstreetmap/josm/io/XmlWriter.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/io/XmlWriter.java

@@ -58 +58 @@
     public static String encode(String unencoded, boolean keepApos) {
         StringBuilder buffer = null;
-        for (int i = 0; i < unencoded.length(); ++i) {
-            String encS = null;
-            if (!keepApos || unencoded.charAt(i) != '\'') {
-                encS = XmlWriter.encoding.get(unencoded.charAt(i));
-            }
-            if (encS != null) {
-                if (buffer == null) {
-                    buffer = new StringBuilder(unencoded.substring(0, i));
+        if (unencoded != null) {
+            for (int i = 0; i < unencoded.length(); ++i) {
+                String encS = null;
+                if (!keepApos || unencoded.charAt(i) != '\'') {
+                    encS = XmlWriter.encoding.get(unencoded.charAt(i));
                 }
-                buffer.append(encS);
-            } else if (buffer != null) {
-                buffer.append(unencoded.charAt(i));
+                if (encS != null) {
+                    if (buffer == null) {
+                        buffer = new StringBuilder(unencoded.substring(0, i));
+                    }
+                    buffer.append(encS);
+                } else if (buffer != null) {
+                    buffer.append(unencoded.charAt(i));
+                }
             }
         }