Re: [PATCH 1/2] clang-format: do not allow short enums on a single line

From: Nathan Chancellor
Date: Tue Feb 11 2025 - 13:13:29 EST


On Mon, Feb 10, 2025 at 09:21:00AM +0100, Michael Riesch wrote:
> clang-format 11 introduced the option "AllowShortEnumsOnASingleLine".
> Set it to "false" in order to avoid short enums of the form:
>
> enum my_short_num { green, red };
>
> With the option set to "false", they will be converted to
>
> enum my_short_num {
> green,
> red
> };
>
> which is consistent with the definition of enums with more items.
>
> Signed-off-by: Michael Riesch <michael.riesch@xxxxxxxxxxxxxx>

Seems reasonable to me.

Reviewed-by: Nathan Chancellor <nathan@xxxxxxxxxx>

> ---
> .clang-format | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/.clang-format b/.clang-format
> index fe1aa1a30d40..8040b516185b 100644
> --- a/.clang-format
> +++ b/.clang-format
> @@ -19,6 +19,7 @@ AlignTrailingComments: false
> AllowAllParametersOfDeclarationOnNextLine: false
> AllowShortBlocksOnASingleLine: false
> AllowShortCaseLabelsOnASingleLine: false
> +AllowShortEnumsOnASingleLine: false
> AllowShortFunctionsOnASingleLine: None
> AllowShortIfStatementsOnASingleLine: false
> AllowShortLoopsOnASingleLine: false
>
> --
> 2.34.1
>