The patch looks good to me. Thank you!
On Sun, Dec 30, 2012 at 8:19 PM, Sebastian Redl < [email protected]> wrote: > > 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 >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
