Summary: | CSSSelector::selectorText should use StringBuilder instead of += | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Oliver Varga <voliver> | ||||
Component: | CSS | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | abecsi, darin, morrita, rhodovan.u-szeged, shinyak, zherczeg | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Oliver Varga
2011-08-05 02:14:13 PDT
Created attachment 105583 [details]
Patch
Comment on attachment 105583 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=105583&action=review > Source/WebCore/css/CSSSelector.cpp:629 > if (CSSSelector* tagHistory = cs->tagHistory()) { How about just to return and eliminate temporal "str" ? Also, you can do early return by checking tagHistory != NULL. Anyway, is it possible to eliminate whole "+" by moving the last block to the head of the function? |