Improve string efficiency using StringBuilder and StringOperations
Created attachment 161156 [details] Patch
Comment on attachment 161156 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=161156&action=review Looks reasonable. AppleConvertedSpace is confusing. > Source/WebCore/css/CSSLineBoxContainValue.cpp:60 > + text.append(' '); We should rename .append(UChar) to .appendUChar() or similar so that my brain can think simple thoughts like "append is slow".... here it's obviously not. :)
> We should rename .append(UChar) to .appendUChar() or similar so that my brain can think simple thoughts like "append is slow".... here it's obviously not. :) We should get rid of String::operator+=() and String::append() so that we can forget that "append is slow" :)
Comment on attachment 161156 [details] Patch Clearing flags on attachment: 161156 Committed r126984: <http://trac.webkit.org/changeset/126984>
All reviewed patches have been landed. Closing bug.
Looking at these patches and seeing how attractive people find +=, Iām wondering if we should add a += operator to StringBuilder.