LibRegex: Avoid using GenericLexer::consume() when at eof

Fixes #10027.
Author: https://github.com/alimpfard Commit: 8e3fe80c06 Pull-request: https://github.com/SerenityOS/serenity/pull/10030 Issue: https://github.com/SerenityOS/serenity/issues/10027 Reviewed-by: https://github.com/bgianfo ✅
2025-08-09 09:39:39 +00:00 · 2021-09-14 19:44:38 +04:30 · 2021-09-14 19:44:38 +04:30 · 8e3fe80c06 · 2024-07-18 03:57:41 +09:00
commit 8e3fe80c06
parent 047c013be0
1 changed files with 4 additions and 1 deletions
--- a/Userland/Libraries/LibRegex/RegexParser.cpp
+++ b/Userland/Libraries/LibRegex/RegexParser.cpp
@ -2229,6 +2229,7 @@ size_t ECMA262Parser::ensure_total_number_of_capturing_parenthesis()
            }
            break;
        case '(':
+            lexer.consume();
            if (lexer.consume_specific('?')) {
                // non-capturing group '(?:', lookaround '(?<='/'(?<!', or named capture '(?<'
                if (!lexer.consume_specific('<'))
@ -2242,8 +2243,10 @@ size_t ECMA262Parser::ensure_total_number_of_capturing_parenthesis()
                ++count;
            }
            break;
+        default:
+            lexer.consume();
+            break;
        }
-        lexer.consume();
    }

    m_total_number_of_capturing_parenthesis = count;