[all-commits] [llvm/llvm-project] e67359: [clang-format] PR50326 AlignAfterOpenBracket Alwa...
MyDeveloperDay via All-commits
all-commits at lists.llvm.org
Thu Jun 3 23:17:07 PDT 2021
Branch: refs/heads/release/12.x
Home: https://github.com/llvm/llvm-project
Commit: e673593742e7dc1f4141d0b6391b995d5152fa6c
https://github.com/llvm/llvm-project/commit/e673593742e7dc1f4141d0b6391b995d5152fa6c
Author: mydeveloperday <mydeveloperday at gmail.com>
Date: 2021-06-03 (Thu, 03 Jun 2021)
Changed paths:
M clang/lib/Format/TokenAnnotator.cpp
M clang/unittests/Format/FormatTest.cpp
Log Message:
-----------
[clang-format] PR50326 AlignAfterOpenBracket AlwaysBreak does not keep to the ColumnLimit
https://bugs.llvm.org/show_bug.cgi?id=50326
{D93626} caused a regression in terms of formatting a function ptr, incorrectly thinking it was a C-Style cast.
This cased a formatter regression between clang-format-11 and clang-format-12
```
void bar()
{
size_t foo = function(Foooo, Barrrrr, Foooo, Barrrr, FoooooooooLooooong);
size_t foo = function(
Foooo, Barrrrr, Foooo, Barrrr, FoooooooooLooooong, BarrrrrrrrrrrrLong,
FoooooooooLooooong);
size_t foo = (*(function))(Foooo, Barrrrr, Foooo, FoooooooooLooooong);
size_t foo = (*(
function))(Foooo, Barrrrr, Foooo, Barrrr, FoooooooooLooooong,
BarrrrrrrrrrrrLong, FoooooooooLooooong);
}
```
became
```
void bar()
{
size_t foo1 = function(Foooo, Barrrrr, Foooo, Barrrr, FoooooooooLooooong);
size_t foo2 = function(
Foooo, Barrrrr, Foooo, Barrrr, FoooooooooLooooong, BarrrrrrrrrrrrLong,
FoooooooooLooooong);
size_t foo3 = (*(function))(Foooo, Barrrrr, Foooo, FoooooooooLooooong);
size_t foo4 = (*(
function))(Foooo, Barrrrr, Foooo, Barrrr, FoooooooooLooooong, BarrrrrrrrrrrrLong, FoooooooooLooooong);
}
```
This fixes this issue by simplify the clause to be specific about what is wanted rather than what is not.
Reviewed By: curdeius, HazardyKnusperkeks
Differential Revision: https://reviews.llvm.org/D102392
(cherry picked from commit eae445f65d077304703e3290ddb4ff28f6d65ff4)
More information about the All-commits
mailing list