RESOLVED FIXED 125254
check-webkit-style: false positive warning for indentation of #ifdef code 
https://bugs.webkit.org/show_bug.cgi?id=125254
Summary check-webkit-style: false positive warning for indentation of #ifdef code
Daniel Bates
Reported 2013-12-04 14:19:53 PST
When processing the attachment #218443 [details] (bug #125239), check-webkit-style emitted the following false positives: ERROR: Source/WebCore/rendering/RenderLayer.cpp:185: Wrong number of spaces before statement. (expected: 4) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderLayer.cpp:2725: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderLayer.cpp:2725: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderMenuList.cpp:81: Wrong number of spaces before statement. (expected: 4) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderLayerCompositor.cpp:432: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderLayerCompositor.cpp:433: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderLayerCompositor.cpp:435: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderLayerCompositor.cpp:437: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderLayerCompositor.cpp:446: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderLayerCompositor.cpp:447: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderFileUploadControl.cpp:106: Wrong number of spaces before statement. (expected: 8) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderFileUploadControl.cpp:107: Wrong number of spaces before statement. (expected: 8) [whitespace/indent] [4] Notice that all of these errors are with respect to code that is within a #ifdef/#else-block.
Attachments
patch (3.00 KB, patch)
2013-12-06 01:54 PST, Gergő Balogh 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Gergő Balogh
Comment 1 2013-12-05 02:08:08 PST
The commit 251bfc338f7f629cc2e3646bd0af707314bddb04 fix most of these issues. I will check the rest of them. Remaining: ERROR: Source/WebCore/rendering/RenderLayer.cpp:180: Wrong number of spaces before statement. (expected: 4) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderLayer.cpp:185: Wrong number of spaces before statement. (expected: 4) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderLayer.cpp:196: Wrong number of spaces before statement. (expected: 4) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderLayer.cpp:199: Wrong number of spaces before statement. (expected: 4) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderLayer.cpp:3640: Wrong number of spaces before statement. (expected: 32) [whitespace/indent] [4] Checked on: commit a53256abaefeacb27f386aa7fd739b81636bc334
Gergő Balogh
Comment 2 2013-12-06 01:54:39 PST
Created attachment 218578 [details] patch
WebKit Commit Bot
Comment 3 2013-12-06 10:52:18 PST
Comment on attachment 218578 [details] patch Clearing flags on attachment: 218578 Committed r160229: <http://trac.webkit.org/changeset/160229>
WebKit Commit Bot
Comment 4 2013-12-06 10:55:55 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.