[PATCH] D155529: [clang-format] Add SpaceInParensOption for __attribute__ keyword

Owen Pan via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 17 17:32:23 PDT 2023


owenpan added inline comments.


================
Comment at: clang/include/clang/Format/Format.h:4214-4220
+    /// Put a space in parentheses around the outside of attribute specifier
+    /// lists.
+    /// \code
+    ///    true:                                  false:
+    ///    __attribute__( (noreturn) )    vs.     __attribute__((noreturn))
+    /// \endcode
+    bool InAttributeSpecifiers;
----------------
I'm against this as the double parens, IMO, should never be separated.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155529



More information about the cfe-commits mailing list