On 30.12.2012, at 19:56, Dmitri Gribenko wrote:

> 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'.

Ah, that's fine then. I was just surprised that inline namespaces would be 
treated differently than normal ones.

Sebastian
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to