Bug 264759

Summary: Follow-up to 270645@main to address review feedback
Product: WebKit Reporter: Chris Dumez <cdumez>
Component: JavaScriptCoreAssignee: Chris Dumez <cdumez>
Status: RESOLVED FIXED    
Severity: Normal CC: darin, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   

Description Chris Dumez 2023-11-13 11:42:45 PST 
Follow-up to 270645@main to address review feedback from Darin. Use Vector constructor / WTF::map() when possible instead of an append*() function.

There are a couple of cases in 270645@main where I keep using append*() even though they were calling reserveInitialCapacity() before because the vectors in question have an inline buffer and doing a Vector move assignment may end up having to copy the values in the inline buffers over.
Comment 1 Chris Dumez 2023-11-13 11:45:40 PST 
Pull request: https://github.com/WebKit/WebKit/pull/20431
Comment 2 EWS 2023-11-13 15:05:37 PST 
Committed 270679@main (1dfe581ad7a8): <https://commits.webkit.org/270679@main>

Reviewed commits have been landed. Closing PR #20431 and removing active labels.
Comment 3 Radar WebKit Bug Importer 2023-11-13 15:06:24 PST 
<rdar://problem/118358370>