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 | 7006e7ebc0098b7627bd3cc13367ea576c25dcbb (patch) | |
| tree | 8803193ad84cfd24a501508c982b002c95977159 /lib/Format/Format.cpp | |
| parent | 4bfc65aab5408f76dfe38e3f4e4375fadeda3bc3 (diff) | |
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));
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@173257 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Format/Format.cpp')
| -rw-r--r-- | lib/Format/Format.cpp | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/lib/Format/Format.cpp b/lib/Format/Format.cpp index 2d9bd2cc06..c75b8ef834 100644 --- a/lib/Format/Format.cpp +++ b/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; |