[PATCH] D32480: [clang-format] Add BinPackNamespaces option

Francois Ferrand via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 17 08:04:07 PDT 2017


Typz added inline comments.


================
Comment at: include/clang/Format/Format.h:153
+  /// \endcode
+  bool AllowSemicolonAfterNamespace;
+
----------------
djasper wrote:
> While I am not entirely opposed to this feature, I think it should be a separate patch.
I totally agree, which is why I created 2 commits; however, since they modify the same code (and I am new to Phabricator) I could not find a way to upload them separately.

Is there a way? Or should I upload one, and upload the next one once the first one has been accepted?


https://reviews.llvm.org/D32480





More information about the cfe-commits mailing list