WebCore/platform/qt/ClipboardQt.cpp:32: Alphabetical sorting problem. [build/include_order] [4] WebCore/platform/qt/ClipboardQt.cpp:42: Alphabetical sorting problem. [build/include_order] [4] WebCore/platform/qt/ClipboardQt.cpp:52: Alphabetical sorting problem. [build/include_order] [4] WebCore/platform/qt/ClipboardQt.cpp:229: Should have a space between // and comment [whitespace/comments] [4] WebCore/platform/qt/ClipboardQt.cpp:236: Declaration has space between type name and * in QPixmap *pixmap [whitespace/declaration] [3] Total errors found: 5 in 1 files
Created attachment 57162 [details] Patch that fixes the coding style issues
QtTriaged
Comment on attachment 57162 [details] Patch that fixes the coding style issues > No new tests. (OOPS!) Ditto.
Created attachment 57957 [details] Patch
Comment on attachment 57957 [details] Patch What's this for? // WebCore::writeURL(m_writableDataObject.get(), url, title, true, false); generally we don't commit commented-out code, it's possible that line should simply be removed.
Comment on attachment 57957 [details] Patch Clearing flags on attachment: 57957 Committed r60905: <http://trac.webkit.org/changeset/60905>
All reviewed patches have been landed. Closing bug.