search

llvm / llvm-project

The LLVM Project is a collection of modular and reusable compiler and toolchain technologies.
http://llvm.org
Other
29.17k stars 12.03k forks source link

AlignConsecutiveAssignments and AlignConsecutiveDeclarations enhancement #37037

Open llvmbot opened 6 years ago

llvmbot commented 6 years ago
Bugzilla Link 37689
Version 6.0
OS Windows NT
Reporter LLVM Bugzilla Contributor

Extended Description

Hi,

It would be great to allow some kind of penalty or a maximum of white space allowed to align and/or align similar C++ type.

Like here

  void PeakingEQCoefficientsOrfanidis(double                coeffs[6],
                                      float                 sr,
                                      double                freq,
                                      double                dBGain,
                                      double                Q,
                                      Biquad::BandwidthMode mode);

most people would only align double/float and leaves the mode alone.

Open to suggestion for the best way to improve this behavior.

Thanks !

mydeveloperday commented 2 years ago

most people would only align double/float

I don't think we can use most here in this sentence, with the number of users we have I think at best we can say like half or slight more of the people.... but the other half...

This would have to be a new option.