-
Notifications
You must be signed in to change notification settings - Fork 312
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
fix: remove use of deprecated apis #1488
Conversation
6d4bbec
to
561e372
Compare
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.
LGTM -- will need some small tweaks to deno_core to land this there, I think.
Another way is by calling a constructor created from a |
@0f-0b thanks for the pointer |
561e372
to
c053eba
Compare
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.
LGTM
5836af3
to
df8f589
Compare
df8f589
to
a770a1b
Compare
3081a5f
to
2a65ecb
Compare
2a65ecb
to
7813647
Compare
fn empty( | ||
_scope: &mut v8::HandleScope, | ||
_args: v8::FunctionCallbackArguments, | ||
_rv: v8::ReturnValue, | ||
) { | ||
} |
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.
Maybe increment a counter here to ensure it's called the correct number of times?
#pragma clang diagnostic ignored "-Wdeprecated-declarations" | ||
|
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.
👍
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.
Looks good to me!
Fixes: #1478
https://docs.google.com/document/d/1FWnrKxjl0P-jY0CMPlzbXkH8xkKoX6nM4e2OXq1EGHU
tldr some* v8 objects now directly store a "wrapped pointer" field which is traced by cppgc. cppgc no longer requires embedder field information when being set up, and this field is set using
Object::wrap
/Object::unwrap
.* objects must use the JSObjectWithEmbedderSlots map