[PATCH] D55170: [clang-format]: Add NonEmptyParentheses spacing option

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Mar 17 15:21:35 PDT 2019


MyDeveloperDay added inline comments.


================
Comment at: include/clang/Format/Format.h:1578
   /// \endcode
   bool SpaceBeforeCpp11BracedList;
 
----------------
boolean here not enum see comment below


================
Comment at: lib/Format/TokenAnnotator.cpp:2608
+        (Style.SpaceBeforeCpp11BracedList == FormatStyle::SBBLO_Always ||
+         (Style.SpaceBeforeCpp11BracedList == FormatStyle::SBBLO_NonEmpty &&
+          Right.ParameterCount > 0)))
----------------
Did the patch upload correctly? this is still not showing a bool for SpaceBeforeCpp11BrackedList


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55170/new/

https://reviews.llvm.org/D55170





More information about the cfe-commits mailing list