Skip to content

Improve RegExp.prototype.compile#2971

Merged
rerobika merged 1 commit intojerryscript-project:masterfrom
rerobika:regexp_compile
Jul 17, 2019
Merged

Improve RegExp.prototype.compile#2971
rerobika merged 1 commit intojerryscript-project:masterfrom
rerobika:regexp_compile

Conversation

@rerobika
Copy link
Copy Markdown
Member

This patch fixes the return value of the routine also fixes several incorrect regression-tests
due to the previously incorrect [[RegExpMatcher]] internal slot check.

JerryScript-DCO-1.0-Signed-off-by: Robert Fancsik frobert@inf.u-szeged.hu

@rerobika rerobika added bug Undesired behaviour ecma builtins Related to ECMA built-in routines labels Jul 12, 2019
Copy link
Copy Markdown
Member

@dbatyai dbatyai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Comment thread tests/jerry/regression-test-issue-1065.js Outdated
This patch fixes the return value of the routine also fixes several incorrect regression-tests
due to the previously incorrect [[RegExpMatcher]] internal slot check.

JerryScript-DCO-1.0-Signed-off-by: Robert Fancsik frobert@inf.u-szeged.hu
Copy link
Copy Markdown
Contributor

@LaszloLango LaszloLango left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rerobika rerobika merged commit a21a419 into jerryscript-project:master Jul 17, 2019
@rerobika rerobika deleted the regexp_compile branch June 16, 2020 08:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Undesired behaviour ecma builtins Related to ECMA built-in routines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants