Subject: Re: [PATCH v4 3/7] clang-format: exclude control macros from SpaceBeforeParens

From: Junio C Hamano <[email-masked]>
Date: Sat, 27 Sep 2025 08:40:15 -0700

Justin Tobler <[email-masked]> writes:

> The formatter currently suggests adding a space between a control macro
> and parentheses. In the Git project, this is not typically expected. Set
> `SpaceBeforeParens` to `ControlStatementsExceptControlMacros`
> accordingly.
>
> Helped-by: Karthik Nayak <[email-masked]>
> Signed-off-by: Justin Tobler <[email-masked]>
> ---
>  .clang-format | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

While this may be a welcome addition, I somehow do not think it
belongs as [3/7] to this series, whose theme is about "git repo
stats".

Perhaps make it a separate topic and have it graduate sooner?

> diff --git a/.clang-format b/.clang-format
> index dcfd0aad60..86b4fe33e5 100644
> --- a/.clang-format
> +++ b/.clang-format
> @@ -149,7 +149,7 @@ SpaceBeforeCaseColon: false
>  #     f();
>  #   }
>  # }
> -SpaceBeforeParens: ControlStatements
> +SpaceBeforeParens: ControlStatementsExceptControlMacros
>  
>  # Don't insert spaces inside empty '()'
>  SpaceInEmptyParentheses: false