WebCore/bridge/qt/qt_runtime.h:20: #ifndef header guard has wrong style, please use: qt_runtime_h [build/header_guard] [5] WebCore/bridge/qt/qt_runtime.h:76: This { should be at the end of the previous line [whitespace/braces] [4] WebCore/bridge/qt/qt_runtime.h:98: This { should be at the end of the previous line [whitespace/braces] [4] WebCore/bridge/qt/qt_runtime.h:170: This { should be at the end of the previous line [whitespace/braces] [4] WebCore/bridge/qt/qt_runtime.h:181: d_func is incorrectly named. Don't use underscores in your identifier names. [readability/naming] [4] WebCore/bridge/qt/qt_runtime.h:193: This { should be at the end of the previous line [whitespace/braces] [4] WebCore/bridge/qt/qt_runtime.h:195: Extra space before ) [whitespace/parens] [2] WebCore/bridge/qt/qt_runtime.h:202: d_func is incorrectly named. Don't use underscores in your identifier names. [readability/naming] [4] WebCore/bridge/qt/qt_runtime.h:213: This { should be at the end of the previous line [whitespace/braces] [4] Total errors found: 9 in 1 files
Created attachment 57108 [details] Patch that fixes the coding style issues
*** Bug 39747 has been marked as a duplicate of this bug. ***
Created attachment 57112 [details] Patch that fixes the coding style issues
QtTriaged
Is this waiting for review? Maybe Anders forgot the flag...
Anders, what is the status of this?
There is no activity on this bug from a long time. So can I assign it to myself and take it further?
(In reply to comment #7) > There is no activity on this bug from a long time. So can I assign it to myself and take it further? Given that Noam is currently rewriting this code, I'd suggest you sync with him (noamr on #qtwebkit) :)
*** This bug has been marked as a duplicate of bug 60842 ***