-
-
Notifications
You must be signed in to change notification settings - Fork 407
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor binding declarations #2887
Conversation
Test262 conformance changes
Fixed tests (296):
Broken tests (291):
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks really nice! It's just missing some links to the corresponding spec and I have a small suggestion to reduce code duplication.
4e54ee4
to
56be789
Compare
I moved all of the I also managed to get the code properly deduplicated, matching the spec. |
Codecov Report
@@ Coverage Diff @@
## main #2887 +/- ##
==========================================
+ Coverage 51.30% 51.76% +0.46%
==========================================
Files 427 428 +1
Lines 42612 43122 +510
==========================================
+ Hits 21860 22324 +464
- Misses 20752 20798 +46
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work! Very nicely documented and the separate instantiation functions makes the code a lot nicer :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Amazing work @raskad! Looks perfect to me! :)
This Pull Request changes the following:
var
bindings in runtime environments withundefined
#2860 as it is not needed anymore.configurable_globals
flag that was passed trough many bytecompiler functions.This change fixes and breaks some annexB tests, but as we do not implement the relevant annexB sections yet, we can ignore those for now. More importantly, this change should make it possible to enable support for these annexB features more easily.