diff options
author | Jo-Philipp Wich <jo@mein.io> | 2021-05-11 17:23:24 +0200 |
---|---|---|
committer | Jo-Philipp Wich <jo@mein.io> | 2021-05-11 17:34:33 +0200 |
commit | f20b56ff64b3144d618d228fbe1a3d07aad96450 (patch) | |
tree | a58b582fe73176ac39f1a3a6c2616f800bed0ff4 /compiler.c | |
parent | 5c4e1ea8dca744482879dfc15ed21fc24beb2775 (diff) |
compiler: properly parse slashes in parenthesized division expressions
Due to the special code path parsing the leading label portion of a
parenthesized expression, slashes following a label were improperly
treated as regular expression literal delimitters, emitting a syntax
error when an otherwise valid expression such as `a / 1` was being
parsed as first sub expression of a parenthesized expression.
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Diffstat (limited to 'compiler.c')
-rw-r--r-- | compiler.c | 10 |
1 files changed, 8 insertions, 2 deletions
@@ -1153,11 +1153,15 @@ uc_compiler_compile_paren(uc_compiler *compiler, bool assignable) /* First try to parse a complete parameter expression and remember the * consumed label tokens as we go. */ while (true) { - if (uc_compiler_parse_match(compiler, TK_LABEL)) { + if (uc_compiler_parse_check(compiler, TK_LABEL)) { if (!varnames) varnames = ucv_array_new(NULL); - ucv_array_push(varnames, ucv_get(compiler->parser->prev.uv)); + ucv_array_push(varnames, ucv_get(compiler->parser->curr.uv)); + + /* A subsequent slash cannot be a regular expression literal */ + compiler->parser->lex.no_regexp = true; + uc_compiler_parse_advance(compiler); } else if (uc_compiler_parse_match(compiler, TK_ELLIP)) { uc_compiler_parse_consume(compiler, TK_LABEL); @@ -1167,6 +1171,8 @@ uc_compiler_compile_paren(uc_compiler *compiler, bool assignable) ucv_array_push(varnames, ucv_get(compiler->parser->prev.uv)); + /* A subsequent slash cannot be a regular expression literal */ + compiler->parser->lex.no_regexp = true; uc_compiler_parse_consume(compiler, TK_RPAREN); maybe_arrowfn = true; |