<div dir="ltr">Would it be possible to send the patch through <a href="http://reviews.llvm.org">reviews.llvm.org</a>? It makes review much easier for me.<div class="gmail_extra"><br><div class="gmail_quote">On Thu, Aug 6, 2015 at 6:18 PM, chris beck via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank" class="cremed">cfe-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div><div><div><div><div>Rationale:<br><br></div>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:<br><br></div>namespace foo { class bar; }<br></div>namespace baz { struct quaz; }<br>...<br><br></div>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).<br><br></div>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.<br><br></div>Best Regards,<br></div>Chris Beck</div>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org" target="_blank" class="cremed">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank" class="cremed">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div></div>