<div style="font-family: arial, helvetica, sans-serif; font-size: 10pt"><div dir="ltr"><div class="gmail_default" style>The patch looks good to me. Thank you!</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Sun, Dec 30, 2012 at 8:19 PM, Sebastian Redl <span dir="ltr"><<a href="mailto:sebastian.redl@getdesigned.at" target="_blank">sebastian.redl@getdesigned.at</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im"><br>
On 30.12.2012, at 19:56, Dmitri Gribenko wrote:<br>
<br>
> On Sun, Dec 30, 2012 at 8:52 PM, Sebastian Redl<br>
> <<a href="mailto:sebastian.redl@getdesigned.at">sebastian.redl@getdesigned.at</a>> wrote:<br>
>><br>
>> On 30.12.2012, at 19:41, Dmitri Gribenko wrote:<br>
>><br>
>>> Hello,<br>
>>><br>
>>> The attached patch teaches the formatter about inline namespaces.<br>
>>> This changes formatting from:<br>
>>><br>
>>> inline namespace X {<br>
>>> class A {<br>
>>> };<br>
>>> }<br>
>>><br>
>>> to:<br>
>>><br>
>>> inline namespace X {<br>
>>> class A {<br>
>>> };<br>
>>> }<br>
>><br>
>> Seems like a strange coding convention. Do we really want this?<br>
><br>
> This is how we currently format namespaces.  This patch is not about<br>
> whether we want indentation or not, it is about treating 'inline<br>
> namespace X' like a plain 'namespace X'.<br>
<br>
</div>Ah, that's fine then. I was just surprised that inline namespaces would be treated differently than normal ones.<br>
<span class="HOEnZb"><font color="#888888"><br>
Sebastian<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br></div></div>