Improve coverage of Regex by removing default case.

a28e4bef · Milo Yip · ba0a137b · a28e4bef
Commit a28e4bef authored Apr 17, 2016 by Milo Yip
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 4 deletions

regex.h include/rapidjson/internal/regex.h +2 -4

No files found.
--- a/include/rapidjson/internal/regex.h
+++ b/include/rapidjson/internal/regex.h
@@ -413,7 +413,8 @@ private:
                }
                return false;

-            case kOneOrMore:
+            default: 
+                RAPIDJSON_ASSERT(op == kOneOrMore);
                if (operandStack.GetSize() >= sizeof(Frag)) {
                    Frag e = *operandStack.template Pop<Frag>(1);
                    SizeType s = NewState(kRegexInvalidState, e.start, 0);
@@ -422,9 +423,6 @@ private:
                    return true;
                }
                return false;
-
-            default:
-                return false;
        }
    }