Reviewed by Darin.
authorap <ap@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 31 Oct 2007 08:20:39 +0000 (08:20 +0000)
committerap <ap@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 31 Oct 2007 08:20:39 +0000 (08:20 +0000)
commit88dedc667558d96b52af6cba744702f05f6e366d
tree81ef0724cfb473efeeecbd0e61162544927b6b59
parentcd7348d187b2660a0d49efe0e6324684ad1fdb9f
    Reviewed by Darin.

        http://bugs.webkit.org/show_bug.cgi?id=10818
        String::append does 2 full copies instead of 1 (or zero!)

        No change in functionality, thus no test.

        * platform/String.cpp:
        (WebCore::String::append): Rewrote to copy once. Also removed an ancient
        FIXME that doesn't seem to make any sense. Note that append() behavior doesn't
        match documented String behavior ("modifications to one instance will
        also modify all others"), but there are a lot of methods that don't.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@27300 268f45cc-cd09-0410-ab3c-d52691b4dbfc
WebCore/ChangeLog
WebCore/platform/String.cpp