diff options
author | Daniel Jasper <djasper@google.com> | 2013-01-23 12:27:43 +0000 |
---|---|---|
committer | Daniel Jasper <djasper@google.com> | 2013-01-23 12:27:43 +0000 |
commit | 43b6548ce0d89c81dcd6f9cab93963daebaf7dba (patch) | |
tree | bde6d5497bbf5b23fb1cb27e7100942a524eafb1 /clang/lib/Format | |
parent | 5b49f47835e89ffc6dbf8468322febe66ebb6ecf (diff) | |
download | bcm5719-llvm-43b6548ce0d89c81dcd6f9cab93963daebaf7dba.tar.gz bcm5719-llvm-43b6548ce0d89c81dcd6f9cab93963daebaf7dba.zip |
Removing the penalty for breaking after "=".
Having seen more cases, this actually was not a good thing to do in the
first place. We can still improve on what we do now, but breaking after
the "=" is good in many cases.
Before:
aaaaaaaaaaaaa = aa->aaaaaaaaaaaaaaaaaaaa(
aaaaaaaaaaaaaaaaaaaaaaaa(aaaaaaaaaaaaaa));
After:
aaaaaaaaaaaaa =
aa->aaaaaaaaaaaaaaaaaaaa(aaaaaaaaaaaaaaaaaaaaaaaa(aaaaaaaaaaaaaa));
llvm-svn: 173257
Diffstat (limited to 'clang/lib/Format')
-rw-r--r-- | clang/lib/Format/Format.cpp | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/clang/lib/Format/Format.cpp b/clang/lib/Format/Format.cpp index 2d9bd2cc061..c75b8ef834b 100644 --- a/clang/lib/Format/Format.cpp +++ b/clang/lib/Format/Format.cpp @@ -702,11 +702,6 @@ private: return prec::Assignment; prec::Level Level = getPrecedence(Left); - // Breaking after an assignment leads to a bad result as the two sides of - // the assignment are visually very close together. - if (Level == prec::Assignment) - return 50; - if (Level != prec::Unknown) return Level; |