fix ticket #279, related to date parsing but actually a regexp bug

--HG-- branch : com.mozilla.es4.smlnj extra : convert_revision : 4afcb0d82ded6ab795dcbadde614aa2328e1e056
bkero · Jan 11, 2008 · 5f0cd81 · 5f0cd81
1 parent a47b8e3
commit 5f0cd81
Showing 1 changed file with 12 additions and 4 deletions.
diff --git a/builtins/RegExpCompiler.es b/builtins/RegExpCompiler.es
@@ -214,6 +214,7 @@ package RegExpInternals
 
             switch (peekCharCode()) {
             case 0x29u /* ")" */:
+                skip();
                 return null;
 
             case 0x2Eu /* "." */:
@@ -286,11 +287,17 @@ package RegExpInternals
                 match(")");
                 return new Capturing(d, capno);
 
-            case 0x5Bu /* "[" */:
-                return characterClass();
+            case 0x5Bu /* "[" */: {
+                let m : Matcher = characterClass();
+                skip();
+                return m;
+            }
 
-            case 0x5Cu /* "\\" */:
-                return atomEscape();
+            case 0x5Cu /* "\\" */: {
+                let m : Matcher = atomEscape();
+                skip();
+                return m;
+            }
 
             case 0x5Eu /* "^" */:
             case 0x24u /* "$" */:
@@ -301,6 +308,7 @@ package RegExpInternals
             case 0x7Cu /* "|" */:
                 // case 0x7Du /* "}" */:
                 // case 0x5Du /* "]" */:
+                skip();
                 return null;
 
             default: {