From 3fc0bb779e751de71166c20a365cd5c28c3de8ed Mon Sep 17 00:00:00 2001 From: Daniel Jasper Date: Wed, 9 Jan 2013 10:40:23 +0000 Subject: Only align after assignments on the top level. This fixes llvm.org/PR14870 and we no longer mess up: template void f(); It removes the nice aligment for assignments inside other expressions, but I am not sure those are actually practically relevant. If so, we can fix those later. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@171966 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Format/Format.cpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'lib/Format/Format.cpp') diff --git a/lib/Format/Format.cpp b/lib/Format/Format.cpp index 53dc97abec..1854d70d90 100644 --- a/lib/Format/Format.cpp +++ b/lib/Format/Format.cpp @@ -333,7 +333,9 @@ private: if (!DryRun) replaceWhitespace(Current, 0, Spaces); - if (RootToken.isNot(tok::kw_for) && + // FIXME: Do we need to do this for assignments nested in other + // expressions? + if (RootToken.isNot(tok::kw_for) && ParenLevel == 0 && (getPrecedence(Previous) == prec::Assignment || Previous.is(tok::kw_return))) State.Indent[ParenLevel] = State.Column + Spaces; -- cgit v1.2.3-18-g5258