[webkit-dev] Fwd: Point 3 of the WebKit Style Guidelines (indenting code inside namespaces in headers)

Eric Seidel eric at webkit.org
Wed Oct 7 22:02:19 PDT 2009


Now from the right email address...


---------- Forwarded message ----------
From: Eric Seidel <eseidel at google.com>
Date: Thu, Oct 8, 2009 at 1:02 AM
Subject: Re: [webkit-dev] Point 3 of the WebKit Style Guidelines
(indenting code inside namespaces in headers)
To: David Levin <levin at google.com>
Cc: Maciej Stachowiak <mjs at apple.com>, webkit-dev Development
<webkit-dev at lists.webkit.org>


Did we ever come to resolve about this issue?

Can we turn it into bugs if that's the case?

Or do we need a vote? :)

-eric

On Tue, Sep 22, 2009 at 8:45 PM, David Levin <levin at google.com> wrote:
>
>
> On Tue, Sep 22, 2009 at 5:37 PM, Maciej Stachowiak <mjs at apple.com> wrote:
>>
>> On Sep 22, 2009, at 1:06 PM, David Hyatt wrote:
>>
>>> I had thought that we resolved ages ago that we would no longer be
>>> indenting code inside namespaces in header files, since that just results in
>>> the entire class declaration being pointlessly indented.
>>>
>>> This is point 3 on the page:
>>>
>>> http://webkit.org/coding/coding-style.html
>>>
>>> I'd like to reverse the Right and Wrong examples to fix this.
>>>
>>> Are there any objections to this change?  I know a few months ago, people
>>> agreed (notably Maciej) that there was no longer any point to essentially
>>> indenting the entire file's contents (when we already don't do this in .cpp
>>> files).
>>
>> I like this change (even though it's hard to make Emacs play along).
>
> Is there a way to get Emacs to auto-indent this correctly? :'(
>
>
> _______________________________________________
> webkit-dev mailing list
> webkit-dev at lists.webkit.org
> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
>
>


More information about the webkit-dev mailing list