-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Use WTF::map() in ScriptArguments::getArgumentsAsStrings() instead of uncheckedAppend() #18527
Use WTF::map() in ScriptArguments::getArgumentsAsStrings() instead of uncheckedAppend() #18527
Conversation
EWS run on previous version of this PR (hash ee534c3) |
if (JSC::jsDynamicCast<JSC::ProxyObject*>(argument)) | ||
return "[object Proxy]"_s; | ||
|
||
String result; |
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.
Does this have to be defined here instead of where it is initialized.
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.
Oh, I just moved the code as is but I don't think there is a reason to defined it here. I'll update.
ee534c3
to
6ee2394
Compare
EWS run on current version of this PR (hash 6ee2394) |
β¦ uncheckedAppend() https://bugs.webkit.org/show_bug.cgi?id=262519 Reviewed by Darin Adler. Use WTF::map() in `ScriptArguments::getArgumentsAsStrings()` instead of `uncheckedAppend()`. It is more efficient, now that `uncheckedAppend()` is just an alias to `append()`. * Source/JavaScriptCore/inspector/ScriptArguments.cpp: (Inspector::argumentAsString): (Inspector::ScriptArguments::getArgumentAtIndexAsString const): (Inspector::ScriptArguments::getArgumentsAsStrings const): Canonical link: https://commits.webkit.org/268793@main
6ee2394
to
9152805
Compare
Committed 268793@main (9152805): https://commits.webkit.org/268793@main Reviewed commits have been landed. Closing PR #18527 and removing active labels. |
9152805
6ee2394
π§ͺ wpe-wk2π§ͺ gtk-wk2π§ͺ mac-AS-debug-wk2