-
-
Notifications
You must be signed in to change notification settings - Fork 406
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
[Merged by Bors] - Removing internal deprecated functions #1854
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1854 +/- ##
==========================================
+ Coverage 47.48% 55.55% +8.06%
==========================================
Files 201 199 -2
Lines 16407 17740 +1333
==========================================
+ Hits 7791 9855 +2064
+ Misses 8616 7885 -731
Continue to review full report at Codecov.
|
aedff3f
to
6f4c82a
Compare
Benchmark for 4ea2348Click to view benchmark
|
f0c7011
to
56ce516
Compare
Test262 conformance changesVM implementation
Fixed tests (6):
|
56ce516
to
3ec5fe7
Compare
Benchmark for d8239cbClick to view benchmark
|
Benchmark for cd3dd05Click to view benchmark
|
Benchmark for 213e27fClick to view benchmark
|
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.
Nice! Will this close #577?
Almost, There still some places where more work/refactoring needs to be done. But I wanted make separate PRs for those refactors :) |
bors r+ |
This Pull Request is related to #577 . It changes the following: - Remove `JsValue::set_field` - Remove `JsValue::set_property` - Remove almost all uses of `JsValue::get_field` - Use `.get_v()` instead of `get_field` according to spec in `serialize_json_property` - Remove `Array::new_array()` - Remove `Array::add_to_array_object()`
Build failed: |
- Remove `Array::new_array()` - Remove `JsValue::set_field()` - Remove `JsValue::set_property()` - Remove some uses of JsValue::get_field
3ec5fe7
to
9c28207
Compare
bors r+ |
This Pull Request is related to #577 . It changes the following: - Remove `JsValue::set_field` - Remove `JsValue::set_property` - Remove almost all uses of `JsValue::get_field` - Use `.get_v()` instead of `get_field` according to spec in `serialize_json_property` - Remove `Array::new_array()` - Remove `Array::add_to_array_object()`
Pull request successfully merged into main. Build succeeded: |
Benchmark for b6f9fc8Click to view benchmark
|
This Pull Request is related to #577 .
It changes the following:
JsValue::set_field
JsValue::set_property
JsValue::get_field
.get_v()
instead ofget_field
according to spec inserialize_json_property
Array::new_array()
Array::add_to_array_object()