[Webkit-unassigned] [Bug 33925] New: [check-webkit-style] False positive when the very first line after namespace open brace is modified in a header file

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jan 20 16:29:54 PST 2010


https://bugs.webkit.org/show_bug.cgi?id=33925

           Summary: [check-webkit-style] False positive when the very
                    first line after namespace open brace is modified in a
                    header file
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: PC
        OS/Version: Mac OS X 10.5
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: Tools / Tests
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: dimich at chromium.org
                CC: eric at webkit.org, levin at chromium.org


This triggers "Code inside a namespace should not be indented. 
[whitespace/indent] [4]", even though the bulk of the file was not touched and
conforms to the 'old style' of indenting the content of the namespace.

I think we should not require the change of the whole header file indent if
only a tiny change is made to it. 

See style bot complaining in bug 22725 for the example.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list