Changeset 274 for branches/LogicMail-1.0/LogicMail/src/org/logicprobe/LogicMail/util/StringParser.java

Timestamp:

08/19/08 07:51:07 PM (5 months ago)

Author:

octorian

Message:

Fixes for #96

Files:

• branches/LogicMail-1.0/LogicMail/src/org/logicprobe/LogicMail/util/StringParser.java (modified) (4 diffs)

branches/LogicMail-1.0/LogicMail/src/org/logicprobe/LogicMail/util/StringParser.java

@@ -334 +334 @@
         return buf.toString();
     }
-    
+
     /**
      * Recursively parse a nested paren string.
@@ -359 +359 @@
         boolean inQuote = false;
         while(q < rawText.length()) {
-            if(rawText.charAt(q) == '\"') {
+            if(isQuoteChar(rawText, q, p)) {
                 if(!inQuote) {
                     inQuote = true;
@@ -422 +422 @@
                         }
                     }
-                    if(rawText.charAt(i) == '\"' && !subInQuote) {
+
+                    if(isQuoteChar(rawText, i, q+1) && !subInQuote) {
                         subInQuote = true;
                     }
-                    else if(rawText.charAt(i) == '\"' && subInQuote) {
+                    else if(isQuoteChar(rawText, i, q+1) && subInQuote) {
                         subInQuote = false;
                     }
@@ -454 +455 @@
         }
         return parsedText;
+    }
+
+    /**
+     * Helper method to determine if a character is a quote.
+     */
+    private static boolean isQuoteChar(String rawText, int index, int startIndex)
+    {
+        if(index == startIndex) {
+            return rawText.charAt(index) == '\"';
+        }
+        else if(rawText.charAt(index) == '\"') {
+            if(rawText.charAt(index - 1) == '\\') {
+                if(index - 2 < startIndex || rawText.charAt(index - 2) == '\\') {
+                    return true;
+                }
+                else {
+                    return false;
+                }
+            }
+            else {
+                return true;
+            }
+        }
+        else {
+            return false;
+        }
     }