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

2015-08-16 Thread Chris Beck via cfe-commits
cbeck88 removed rL LLVM as the repository for this revision. cbeck88 updated this revision to Diff 32249. cbeck88 added a comment. - Fixed formatting in header - Fixed handling of nested namespaces - Added unit tests djasper: I thought about it, I guess I don't have a particularly good reason th

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

2015-08-11 Thread Chris Beck via cfe-commits
cbeck88 created this revision. cbeck88 added a reviewer: djasper. cbeck88 added a subscriber: cfe-commits. cbeck88 set the repository for this revision to rL LLVM. Herald added a subscriber: klimek. Rationale: I sometimes use a different clang tool, iwyu ("include what you use"), to clean up hea

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

2015-08-10 Thread Daniel Jasper via cfe-commits
djasper added inline comments. Comment at: include/clang/Format/Format.h:115 @@ -114,1 +114,3 @@ + /// \brief If \c true, namespace a { class b; } can be put + /// on a single line Don't use markup here. Use a \code block instead. Comment a