[clang-format] Add SpaceBeforeUnderscoreParens option for GNU gettext…

clang/docs/ClangFormatStyleOptions.rst

@@ -6605,6 +6605,18 @@ the configuration (without a prefix: ``Auto``).
      int a [5];                    vs.      int a[5];
      int a [5][5];                 vs.      int a[5][5];
 
+.. _SpaceBetweenUnderscoreParens:
+
+**SpaceBetweenUnderscoreParens** (``Boolean``) :versionbadge:`clang-format 19` :ref:`¶ <SpaceBetweenUnderscoreParens>`
+  If ``false``, spaces will be removed between underscore and an opening
+  parenthesis. This is specifically for the gettext macro ``_()`` commonly
+  used in GNU projects.
+
+  .. code-block:: c++
+
+     true:                                  false:
+     _ (message);                   vs.     _(message);
+
 .. _SpaceInEmptyBlock:
 
 **SpaceInEmptyBlock** (``Boolean``) :versionbadge:`clang-format 10` :ref:`¶ <SpaceInEmptyBlock>`

clang/include/clang/Format/Format.h

@@ -4896,6 +4896,16 @@ struct FormatStyle {
   /// \version 10
   // bool SpaceInEmptyBlock;
 
+  /// If ``false``, spaces will be removed between underscore and an opening
+  /// parenthesis. This is specifically for the gettext macro ``_()`` commonly
+  /// used in GNU projects.
+  /// \code
+  ///    true:                                  false:
+  ///    _ (message);                   vs.     _(message);
+  /// \endcode
+  /// \version 19
+  bool SpaceBetweenUnderscoreParens;
+
   /// Style of when to insert a space in empty braces.
   enum SpaceInEmptyBracesStyle : int8_t {
     /// Always insert a space in empty braces.

clang/lib/Format/Format.cpp

@@ -1219,6 +1219,8 @@ template <> struct MappingTraits<FormatStyle> {
                    Style.SpaceBeforeCtorInitializerColon);
     IO.mapOptional("SpaceBeforeInheritanceColon",
                    Style.SpaceBeforeInheritanceColon);
+    IO.mapOptional("SpaceBetweenUnderscoreParens",
+                   Style.SpaceBetweenUnderscoreParens);
     IO.mapOptional("SpaceBeforeJsonColon", Style.SpaceBeforeJsonColon);
     IO.mapOptional("SpaceBeforeParens", Style.SpaceBeforeParens);
     IO.mapOptional("SpaceBeforeParensOptions", Style.SpaceBeforeParensOptions);
@@ -1713,6 +1715,7 @@ FormatStyle getLLVMStyle(FormatStyle::LanguageKind Language) {
   LLVMStyle.SpaceBeforeCpp11BracedList = false;
   LLVMStyle.SpaceBeforeCtorInitializerColon = true;
   LLVMStyle.SpaceBeforeInheritanceColon = true;
+  LLVMStyle.SpaceBetweenUnderscoreParens = true;
   LLVMStyle.SpaceBeforeJsonColon = false;
   LLVMStyle.SpaceBeforeParens = FormatStyle::SBPO_ControlStatements;
   LLVMStyle.SpaceBeforeParensOptions = {};
@@ -2044,6 +2047,7 @@ FormatStyle getGNUStyle() {
   Style.FixNamespaceComments = false;
   Style.KeepFormFeed = true;
   Style.SpaceBeforeParens = FormatStyle::SBPO_Always;
+  Style.SpaceBetweenUnderscoreParens = false;
   return Style;
 }
 

clang/lib/Format/TokenAnnotator.cpp

@@ -4901,6 +4901,11 @@ bool TokenAnnotator::spaceRequiredBetween(const AnnotatedLine &Line,
     // Handle builtins like identifiers.
     if (Line.Type != LT_PreprocessorDirective &&
         (Left.Tok.getIdentifierInfo() || Left.is(tok::r_paren))) {
+      // Check for special case: single underscore token (gettext macro).
+      if (Left.Tok.getIdentifierInfo() && !Style.SpaceBetweenUnderscoreParens &&
+          Left.TokenText == "_") {
+        return false;
+      }
       return spaceRequiredBeforeParens(Right);
     }
     return false;

clang/unittests/Format/FormatTest.cpp

@@ -17852,6 +17852,37 @@ TEST_F(FormatTest, ConfigurableSpacesInParens) {
                Spaces);
 }
 
+TEST_F(FormatTest, SpaceBetweenUnderscoreParens) {
+  FormatStyle Style = getLLVMStyle();
+  Style.SpaceBeforeParens = FormatStyle::SBPO_Always;
+
+  EXPECT_TRUE(Style.SpaceBetweenUnderscoreParens);
+  verifyFormat("func (arg);", Style);
+  verifyFormat("my_func (arg);", Style);
+  verifyFormat("_ (message);", Style);
+  verifyFormat("underscore_ (param);", Style);
+
+  Style.SpaceBetweenUnderscoreParens = false;
+  verifyFormat("func (arg);", Style);
+  verifyFormat("my_func (arg);", Style);
+  verifyFormat("_(message);", Style);
+  verifyFormat("underscore_ (param);", Style);
+  verifyFormat("_private_func (data);", Style);
+
+  FormatStyle GNUStyle = getGNUStyle();
+  EXPECT_FALSE(GNUStyle.SpaceBetweenUnderscoreParens);
+  EXPECT_EQ(GNUStyle.SpaceBeforeParens, FormatStyle::SBPO_Always);
+  verifyFormat("func (arg);", GNUStyle);
+  verifyFormat("my_func (arg);", GNUStyle);
+  verifyFormat("_(message);", GNUStyle);
+  verifyFormat("_private_func (data);", GNUStyle);
+
+  verifyFormat("printf (_(\"Hello\"));\n"
+               "func (arg);\n"
+               "_(\"World\");",
+               GNUStyle);
+}
+
 TEST_F(FormatTest, ConfigurableSpacesInSquareBrackets) {
   verifyFormat("int a[5];");
   verifyFormat("a[3] += 42;");