[Webkit-unassigned] [Bug 119319] New: Don't use Vector<UChar> to build strings

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Jul 30 21:38:55 PDT 2013


https://bugs.webkit.org/show_bug.cgi?id=119319

           Summary: Don't use Vector<UChar> to build strings
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: NEW
          Keywords: BlinkMergeCandidate
          Severity: Normal
          Priority: P2
         Component: Web Template Framework
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: rniwa at webkit.org
                CC: darin at apple.com, ggaren at apple.com, koivisto at iki.fi,
                    benjamin at webkit.org, akling at apple.com,
                    msaboff at apple.com


Consider merging https://chromium.googlesource.com/chromium/blink/+/9fda929af533c3c1aa3961c4818ea30ed7499021

This CL removes a bunch of code sites that were using Vector<UChar> to
construct strings. That used to be a good way to construct strings, but in the
modern world, we have much better alternatives in the form of StringBuffer and
StringBuilder. (You should use StringBuffer if you know exactly how much memory
you need and you should use StringBuilder if you require a variable amount of
memory.)

This CL removes more clients of the StringImpl pointer. There are still a few
left, which require more study.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list