On Sun, Dec 30, 2012 at 8:52 PM, Sebastian Redl
<[email protected]> wrote:
>
> On 30.12.2012, at 19:41, Dmitri Gribenko wrote:
>
>> Hello,
>>
>> The attached patch teaches the formatter about inline namespaces.
>> This changes formatting from:
>>
>> inline namespace X {
>> class A {
>> };
>> }
>>
>> to:
>>
>> inline namespace X {
>> class A {
>> };
>> }
>
> Seems like a strange coding convention. Do we really want this?
This is how we currently format namespaces. This patch is not about
whether we want indentation or not, it is about treating 'inline
namespace X' like a plain 'namespace X'.
And on the coding guidelines side -- yes, I think we do want this.
Imagine a header file:
namespace clang {
...lots of stuff...
} // end namespace clang
Everything in the namespace is usually not indented.
Dmitri
--
main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
(j){printf("%d\n",i);}}} /*Dmitri Gribenko <[email protected]>*/
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
