1405914 - clang-format: Bad restyle of a struct literal

Status: NEW

(Developer Infrastructure :: Lint and Formatting, defect, P3)

Description

[Nicholas Nethercote [inactive]]

clang-format just did this:

> -static const mozilla::Module kPrefModule = {
> -  mozilla::Module::kVersion,
> -  kPrefCIDs,
> -  kPrefContracts,
> -  nullptr,
> -  nullptr,
> -  nullptr,
> -  UnloadPrefsModule
> -};
> +static const mozilla::Module kPrefModule = { mozilla::Module::kVersion,
> +                                             kPrefCIDs,
> +                                             kPrefContracts,
> +                                             nullptr,
> +                                             nullptr,
> +                                             nullptr,
> +                                             UnloadPrefsModule };

Pretty gross. It would be nice if that could be avoided.

Comment 1

[Andi [:andi]]

This is the expected behavior since we use flag |BAS_Align| that according to the clang-format documentation:

>>    /// \brief Align parameters on the open bracket, e.g.:
>>    /// \code
>>    ///   someLongFunction(argument1,
>>    ///                    argument2);
>>    /// \endcode
>>    BAS_Align, 

Also I've looked other our coding style page to see why we use that flag and It seems the explanation can be found here: https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Coding_Style#Methods

However I didn't find where the behavior that you mention is documented, and I don't think that implementing such a patch would be accepted up-stream since it envelopes treating a very particular scenario that doesn't have a wider application.