summaryrefslogtreecommitdiffstats
path: root/clang/lib/Format
diff options
context:
space:
mode:
authorDaniel Jasper <djasper@google.com>2013-01-23 12:27:43 +0000
committerDaniel Jasper <djasper@google.com>2013-01-23 12:27:43 +0000
commit43b6548ce0d89c81dcd6f9cab93963daebaf7dba (patch)
treebde6d5497bbf5b23fb1cb27e7100942a524eafb1 /clang/lib/Format
parent5b49f47835e89ffc6dbf8468322febe66ebb6ecf (diff)
downloadbcm5719-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.cpp5
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;
OpenPOWER on IntegriCloud