[webkit-reviews] review denied: [Bug 97320] Use emptyString instead of String("") : [Attachment 165110] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Sep 21 09:09:10 PDT 2012


Adam Barth <abarth at webkit.org> has denied Patrick R. Gansterer
<paroga at paroga.com>'s request for review:
Bug 97320: Use emptyString instead of String("")
https://bugs.webkit.org/show_bug.cgi?id=97320

Attachment 165110: Patch
https://bugs.webkit.org/attachment.cgi?id=165110&action=review

------- Additional Comments from Adam Barth <abarth at webkit.org>
My understanding is that emptyString is only safe to use on the main thread. 
Many of these lines of code look like they can be reached on worker threads.


More information about the webkit-reviews mailing list