[PATCH] clang-format: Add "AllowShortNamespacesOnASingleLine" option

chris beck via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 6 09:18:00 PDT 2015


Rationale:

I sometimes use a different clang tool, iwyu ("include what you use"), to
clean up header file inclusions in my C++ projects. Iwyu seeks to correct
the includes of a header or cpp unit so that definitions which are needed
are included, and definitions which only need to be forward declared are
forward declared. It often generates code like this at the top of your file:

namespace foo { class bar; }
namespace baz { struct quaz; }
...

Unfortunately, clang-format dislikes braces which are arranged this way and
always wants to break after them, and after the forward declaration, no
matter what configuration options are used (as far as I can tell).

I wrote a small patch so that short namespaces like these can be set on a
single line regardless of chosen brace-style if a boolean option
"AllowShortNamespacesOnASingleLine" is enabled, please find attached.

Best Regards,
Chris Beck
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150806/04cf6bc9/attachment-0001.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ShortNamespaces.patch
Type: text/x-patch
Size: 3414 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150806/04cf6bc9/attachment-0001.bin>


More information about the cfe-commits mailing list