Don't care either. (Just as long as there is consistency and that the style guide reflects the current consensus.) > Will check-webkit-style need an update? Yes it will. (I'm pretty sure that the single fix needed will be in check_namespace_indentation and then any corresponding test cases.)
Dave On Tue, Sep 22, 2009 at 1:24 PM, Eric Seidel <e...@webkit.org> wrote: > I'm fine either way. Will check-webkit-style need an update? I think it > might. > > On Tue, Sep 22, 2009 at 1:23 PM, Sam Weinig <sam.wei...@gmail.com> wrote: > > I also think this change is the right way to go. r=me. > > > > On Tue, Sep 22, 2009 at 1:20 PM, Brady Eidson <beid...@apple.com> wrote: > >> > >> I've always hated the indentation in header files. > >> > >> I have no objections. > >> > >> ~Brady > >> > >> 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). > >>> > >>> dave > >>> (hy...@apple.com) > >>> > >>> _______________________________________________ > >>> webkit-dev mailing list > >>> webkit-dev@lists.webkit.org > >>> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev > >> > >> _______________________________________________ > >> webkit-dev mailing list > >> webkit-dev@lists.webkit.org > >> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev > > > > > > _______________________________________________ > > webkit-dev mailing list > > webkit-dev@lists.webkit.org > > http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev > > > > > _______________________________________________ > webkit-dev mailing list > webkit-dev@lists.webkit.org > http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev >
_______________________________________________ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev