* Fix bug in regexp parser.

regexp2000/src/parser.cc

 // Copyright 2006-2008 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 4046 matching lines @@
 
   ASSERT_EQ(current(), '[');
   Advance();
   bool is_negated = false;
   if (current() == '^') {
     is_negated = true;
     Advance();
   }
   ZoneList<CharacterRange>* ranges = new ZoneList<CharacterRange>(2);
   while (has_more() && current() != ']') {
-    if (current() == '-') {
-      Advance();
-      ranges->Add(CharacterRange::Singleton('-'));
-    } else {
-      bool is_char_class = false;
-      CharacterRange first = ParseClassAtom(&is_char_class, ranges, CHECK_OK);
-      if (!is_char_class) {
-        if (current() == '-') {
-          Advance();
-          CharacterRange next =
-              ParseClassAtom(&is_char_class, ranges, CHECK_OK);
-          if (is_char_class) {
-            return ReportError(CStrVector(kIllegal), CHECK_OK);
-          }
-          if (first.from() > next.to()) {
-            return ReportError(CStrVector(kRangeOutOfOrder), CHECK_OK);
-          }
-          ranges->Add(CharacterRange::Range(first.from(), next.to()));
-        } else {
+    bool is_char_class = false;
+    CharacterRange first = ParseClassAtom(&is_char_class, ranges, CHECK_OK);
+    if (!is_char_class) {
+      if (current() == '-') {
+        Advance();
+        if (current() == ']') {
           ranges->Add(first);
+          ranges->Add(CharacterRange::Singleton('-'));
+          break;
         }
+        CharacterRange next =
+            ParseClassAtom(&is_char_class, ranges, CHECK_OK);
+        if (is_char_class) {
+          return ReportError(CStrVector(kIllegal), CHECK_OK);
+        }
+        if (first.from() > next.to()) {
+          return ReportError(CStrVector(kRangeOutOfOrder), CHECK_OK);
+        }
+        ranges->Add(CharacterRange::Range(first.from(), next.to()));
+      } else {
+        ranges->Add(first);
       }
     }
   }
   if (!has_more()) {
     return ReportError(CStrVector(kUnterminated), CHECK_OK);
   }
   Advance();
   if (ranges->length() == 0) {
     return RegExpEmpty::GetInstance();
   } else {
@@ skipping 116 matching lines @@
                        start_position,
                        is_expression);
   return result;
 }
 
 
 #undef NEW
 
 
 } }  // namespace v8::internal