[webkit-reviews] review denied: [Bug 15907] Implement USE(QT4_UNICODE) for UTF8 -> UTF16 conversion : [Attachment 17129] implements qt4 unicode in c_utility.cpp

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Nov 8 14:58:51 PST 2007


Mark Rowe (bdash) <mrowe at apple.com> has denied Justin Haygood
<jhaygood at reaktix.com>'s request for review:
Bug 15907: Implement USE(QT4_UNICODE) for UTF8 -> UTF16 conversion
http://bugs.webkit.org/show_bug.cgi?id=15907

Attachment 17129: implements qt4 unicode in c_utility.cpp
http://bugs.webkit.org/attachment.cgi?id=17129&action=edit

------- Additional Comments from Mark Rowe (bdash) <mrowe at apple.com>
The patch needs a ChangeLog entry, and the coding style should follow the style
guidelines.  I know that the ICU code above does not, but it'd be great if new
code would.  In particular, the use of C-style casts, and the one-line for
statement with strange whitespace usage.


More information about the webkit-reviews mailing list