diff options
author | Manuel Klimek <klimek@google.com> | 2013-01-02 16:30:12 +0000 |
---|---|---|
committer | Manuel Klimek <klimek@google.com> | 2013-01-02 16:30:12 +0000 |
commit | a080a187fa7e538da3212c7d5e678e4b7ae03253 (patch) | |
tree | 5a2c54db0a6a2ec4e1764cea7c7080d6898b8d84 /lib/Format/UnwrappedLineParser.cpp | |
parent | ef5b9c3d38a1f6d0921591cb2749e529a8cc4a2e (diff) |
Fixes use of unescaped newlines when formatting preprocessor directives.
This is the first step towards handling preprocessor directives. This
patch only fixes the most pressing issue, namely correctly escaping
newlines for tokens within a sequence of a preprocessor directive.
The next step will be to fix incorrect format decisions on #define
directives.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@171393 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Format/UnwrappedLineParser.cpp')
-rw-r--r-- | lib/Format/UnwrappedLineParser.cpp | 24 |
1 files changed, 18 insertions, 6 deletions
diff --git a/lib/Format/UnwrappedLineParser.cpp b/lib/Format/UnwrappedLineParser.cpp index 78a1abdcf8..614125b943 100644 --- a/lib/Format/UnwrappedLineParser.cpp +++ b/lib/Format/UnwrappedLineParser.cpp @@ -80,13 +80,25 @@ bool UnwrappedLineParser::parseBlock(unsigned AddLevels) { } void UnwrappedLineParser::parsePPDirective() { - while (!eof()) { - nextToken(); - if (FormatTok.NewlinesBefore > 0) { - addUnwrappedLine(); - return; - } + assert(FormatTok.Tok.is(tok::hash) && "'#' expected"); + nextToken(); + + Line.InPPDirective = true; + if (FormatTok.Tok.getIdentifierInfo() == NULL) { + addUnwrappedLine(); + Line.InPPDirective = false; + return; } + + do { + if (FormatTok.NewlinesBefore > 0 && + FormatTok.HasUnescapedNewline) { + break; + } + nextToken(); + } while (!eof()); + addUnwrappedLine(); + Line.InPPDirective = false; } void UnwrappedLineParser::parseComments() { |