Use assertion for impossible case(The Predict() can ensure the token …

…is ColonToken, otherwise it would be marked as Error state. So there is no need to check ColonToken again).
ylavic · Apr 15, 2015 · 3993332 · 3993332
1 parent 8576747
commit 3993332
Showing 1 changed file with 3 additions and 6 deletions.
diff --git a/include/rapidjson/reader.h b/include/rapidjson/reader.h
@@ -1273,12 +1273,9 @@ class GenericReader {
                 return dst;
 
         case IterativeParsingKeyValueDelimiterState:
-            if (token == ColonToken) {
-                is.Take();
-                return dst;
-            }
-            else
-                return IterativeParsingErrorState;
+            RAPIDJSON_ASSERT(token == ColonToken);
+            is.Take();
+            return dst;
 
         case IterativeParsingMemberValueState:
             // Must be non-compound value. Or it would be ObjectInitial or ArrayInitial state.