Skip to content

Commit

Permalink
Fix parsing class body that includes stray semicolon after a static b…
Browse files Browse the repository at this point in the history
…lock (jerryscript-project#5145)

JerryScript-DCO-1.0-Signed-off-by: Máté Tokodi mate.tokodi@szteszoftver.hu
  • Loading branch information
matetokodi authored Jun 18, 2024
1 parent 4e89e18 commit 2dbb6f7
Show file tree
Hide file tree
Showing 2 changed files with 25 additions and 6 deletions.
12 changes: 6 additions & 6 deletions jerry-core/parser/js/js-scanner.c
Original file line number Diff line number Diff line change
Expand Up @@ -2700,6 +2700,12 @@ scanner_scan_all (parser_context_t *context_p) /**< context */
identifier_found = true;
}

if (context_p->token.type == LEXER_SEMICOLON)
{
scanner_context.mode = SCAN_MODE_CLASS_BODY;
continue;
}

if (!identifier_found)
{
scanner_raise_error (context_p);
Expand Down Expand Up @@ -2728,12 +2734,6 @@ scanner_scan_all (parser_context_t *context_p) /**< context */
break;
}

if (context_p->token.type == LEXER_SEMICOLON)
{
scanner_context.mode = SCAN_MODE_CLASS_BODY;
continue;
}

if (context_p->token.type != LEXER_RIGHT_BRACE && !(context_p->token.flags & LEXER_WAS_NEWLINE))
{
scanner_raise_error (context_p);
Expand Down
19 changes: 19 additions & 0 deletions tests/jerry/regression-test-issue-5114.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
// Copyright JS Foundation and other contributors, http://js.foundation
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.

class c {
static {
}
;
}

0 comments on commit 2dbb6f7

Please sign in to comment.