aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--lib/Format/UnwrappedLineParser.cpp7
-rw-r--r--lib/Format/UnwrappedLineParser.h3
-rw-r--r--unittests/Format/FormatTest.cpp7
3 files changed, 12 insertions, 5 deletions
diff --git a/lib/Format/UnwrappedLineParser.cpp b/lib/Format/UnwrappedLineParser.cpp
index c1bafa9a3b..2f3a6034bc 100644
--- a/lib/Format/UnwrappedLineParser.cpp
+++ b/lib/Format/UnwrappedLineParser.cpp
@@ -27,10 +27,11 @@ public:
ScopedMacroState(UnwrappedLine &Line, FormatTokenSource *&TokenSource,
FormatToken &ResetToken)
: Line(Line), TokenSource(TokenSource), ResetToken(ResetToken),
- PreviousTokenSource(TokenSource) {
+ PreviousLineLevel(Line.Level), PreviousTokenSource(TokenSource) {
TokenSource = this;
// FIXME: Back up all other state (errors, line indent, etc) and reset after
// parsing the macro.
+ Line.Level = 0;
Line.InPPDirective = true;
}
@@ -38,7 +39,7 @@ public:
TokenSource = PreviousTokenSource;
ResetToken = Token;
Line.InPPDirective = false;
- Line.Level = 0; // FIXME: Test + this is obviously incorrect
+ Line.Level = PreviousLineLevel;
}
virtual FormatToken getNextToken() {
@@ -65,7 +66,7 @@ private:
UnwrappedLine &Line;
FormatTokenSource *&TokenSource;
FormatToken &ResetToken;
-
+ unsigned PreviousLineLevel;
FormatTokenSource *PreviousTokenSource;
FormatToken Token;
diff --git a/lib/Format/UnwrappedLineParser.h b/lib/Format/UnwrappedLineParser.h
index 69ac768325..a8e5b739bc 100644
--- a/lib/Format/UnwrappedLineParser.h
+++ b/lib/Format/UnwrappedLineParser.h
@@ -124,6 +124,9 @@ private:
void nextToken();
void readToken();
+ // FIXME: We are constantly running into bugs where Line.Level is incorrectly
+ // subtracted from beyond 0. Introduce a method to subtract from Line.Level
+ // and use that everywhere in the Parser.
UnwrappedLine Line;
FormatToken FormatTok;
diff --git a/unittests/Format/FormatTest.cpp b/unittests/Format/FormatTest.cpp
index 89fe71c5ba..2165056a7f 100644
--- a/unittests/Format/FormatTest.cpp
+++ b/unittests/Format/FormatTest.cpp
@@ -466,9 +466,12 @@ TEST_F(FormatTest, HashInMacroDefinition) {
" }", getLLVMStyleWithColumns(11));
}
+TEST_F(FormatTest, IndentPreprocessorDirectivesAtZero) {
+ EXPECT_EQ("{\n {\n#define A\n }\n}", format("{{\n#define A\n}}"));
+}
+
// FIXME: write test for unbalanced braces in macros...
-// FIXME: test { { #include "a.h" } }
-// FIXME: test # in the middle of a statement without \n before it
+// FIXME: test # inside a normal statement (like {#define A b})
TEST_F(FormatTest, MixingPreprocessorDirectivesAndNormalCode) {
EXPECT_EQ(