Fixed documentation for the AlignConsecutive* options. I have used the suggested approach: Remove all documentation from AlignConsecutiveStyle and instead put the documentation of the possible values and what they do in the AlignConsecutive* options' docs.

Fixed formatting issue.

In D93986#2489140, @HazardyKnusperkeks wrote:

Apart from my inline comment and the pre merge check this is superb. (I will not accept it, until we have reached a conclusion for the documentation.)

Fix merge conflict in the release notes.

This is now an all-in-one revision, including bit fields, assignments, declarations and macros. I did not reproduce the full "across empty lines, across comments, across empty lines *and* comments" test suite for all four of them, since they all use the same logic (and code, mostly). AlignConsecutiveAssignments is tested extensively, AlignConsecutiveBitFields and AlignConsecutiveDeclarations only have tests for AlignAcrossEmptyLinesAndComments (since they delegate to the same AlignTokens method), and AlignConsecutiveMacros has its own set of tests, since it uses a different implementation (of the same logic, basically).

In D93986#2477860, @HazardyKnusperkeks wrote:

In D93986#2477474, @tinloaf wrote:

In D93986#2477383, @MyDeveloperDay wrote:

Ideally we should add a comment to the release notes, (which you could do via a separate NFC commit if you wanted)

Thank, I'll do that. But before that, I will create a revision that adds the same feature for the other three alignment types.

I think it should happen in this revision so that it is atomically.

• Unify the type of the options for AlignConsecutive{BitFields, Declarations, Assignments, Macros}
• Implement across-alignment for bit fields, declarations, assignments and macros
• Add tests for across-alignment for bit fields, declarations, assignments and macros
• Add changelog entry

In D93986#2477383, @MyDeveloperDay wrote:

Ideally we should add a comment to the release notes, (which you could do via a separate NFC commit if you wanted)

Fix bogus default case reported by clang-tidy.

Address reviewer comments, noteworthy changes:

Address MyDeveloperDay's review comments.

Fix formatting issue resulting from ancient clang-format.

Two remarks, first: I made the change only for AlignConsecutiveAssignments, but I think it should be consistent for AlignConsecutiveMacros, AlignConsecutiveBitFields and AlignConsecutiveDeclarations, as well. If this change is accepted, I will make corresponding changes to the other three.