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
WebScriptObject: Refactor getListFromNSArray to use a for-in loop #11439
WebScriptObject: Refactor getListFromNSArray to use a for-in loop #11439
Conversation
EWS run on previous version of this PR (hash 9c41fdd) |
9c41fdd
to
503914d
Compare
EWS run on previous version of this PR (hash 503914d) |
503914d
to
9f92b20
Compare
EWS run on previous version of this PR (hash 9f92b20) |
9f92b20
to
c796cab
Compare
EWS run on previous version of this PR (hash c796cab) |
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.
Good change. Please remove the braces around what's now a single line block: https://webkit.org/code-style-guidelines/#braces
Done! |
6b53d04
to
1b52e9e
Compare
EWS run on current version of this PR (hash 1b52e9e) |
https://bugs.webkit.org/show_bug.cgi?id=253823 Reviewed by Alexey Proskuryakov. This makes the code cleaner and more terse. * Source\WebCore\bridge\objc\WebScriptObject.mm:(getListFromNSArray): Prefer for-in loop over regular for loop for iteration of an NSArray. Canonical link: https://commits.webkit.org/261607@main
1b52e9e
to
5b1535c
Compare
Committed 261607@main (5b1535c): https://commits.webkit.org/261607@main Reviewed commits have been landed. Closing PR #11439 and removing active labels. |
5b1535c
1b52e9e
π wpeπ wincairoπ§ͺ wpe-wk2π§ͺ ios-wk2π gtkπ§ͺ api-iosπ§ͺ mac-wk1π§ͺ gtk-wk2π tvπ§ͺ api-gtkπ§ͺ mac-AS-debug-wk2π watch