1berry/qtbase
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

remove the entirely insane whitespace compression

Browse Source 
it was merely an artifact of using QString::simplified() on the
unparsed (!) project code. there is no reason why anyone should actually
rely on it, so just remove it.

Change-Id: If9b957c4b1263f3990a2331f8851bb1c06154ea8
Reviewed-by: Qt Doc Bot <qt_docbot@qt-project.org>
Reviewed-by: Joerg Bornemann <joerg.bornemann@digia.com>
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@digia.com>
This commit is contained in:
Oswald Buddenhagen 2012-09-03 17:05:33 +02:00 committed by Qt by Nokia
parent 2084b48a79
commit c3c4f1eb53
1 changed files with 2 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
qmake/library/qmakeparser.cpp
View File

@ -340,9 +340,8 @@ bool QMakeParser::read(ProFile *pro, const QString &in, int line, SubGrammar gra
int argc = 0; int argc = 0;
int wordCount = 0; // Number of words in currently accumulated expression int wordCount = 0; // Number of words in currently accumulated expression
int lastIndent = 0; // Previous line's indentation, to detect accidental continuation abuse int lastIndent = 0; // Previous line's indentation, to detect accidental continuation abuse
bool putSpace = false; // Only ever true inside quoted string
bool lineMarked = true; // For in-expression markers bool lineMarked = true; // For in-expression markers
ushort needSep = TokNewStr; // Complementary to putSpace: separator outside quotes ushort needSep = TokNewStr; // Met unquoted whitespace
ushort quote = 0; ushort quote = 0;
ushort term = 0; ushort term = 0;
@ -494,10 +493,6 @@ bool QMakeParser::read(ProFile *pro, const QString &in, int line, SubGrammar gra
if (c == '$') { if (c == '$') {
if (*cur == '$') { // may be EOF, EOL, WS, '#' or '\\' if past end if (*cur == '$') { // may be EOF, EOL, WS, '#' or '\\' if past end
cur++; cur++;
if (putSpace) {
putSpace = false;
*ptr++ = ' ';
}
FLUSH_LITERAL(); FLUSH_LITERAL();
if (!lineMarked) { if (!lineMarked) {
lineMarked = true; lineMarked = true;
@ -619,13 +614,6 @@ bool QMakeParser::read(ProFile *pro, const QString &in, int line, SubGrammar gra
} else if (quote) { } else if (quote) {
if (c == quote) { if (c == quote) {
quote = 0; quote = 0;
if (putSpace) {
putSpace = false;
*ptr++ = ' ';
}
goto nextChr;
} else if ((c == ' ' || c == '\t') && context != CtxPureValue) {
putSpace = true;
goto nextChr; goto nextChr;
} else if (c == '!' && ptr == xprPtr && context == CtxTest) { } else if (c == '!' && ptr == xprPtr && context == CtxTest) {
m_invert ^= true; m_invert ^= true;
@ -793,10 +781,6 @@ bool QMakeParser::read(ProFile *pro, const QString &in, int line, SubGrammar gra
languageWarning(fL1S("Possible accidental line continuation")); languageWarning(fL1S("Possible accidental line continuation"));
} }
} }
if (putSpace) {
putSpace = false;
*ptr++ = ' ';
}
*ptr++ = c; *ptr++ = c;
nextChr: nextChr:
if (cur == end) if (cur == end)
@ -807,7 +791,7 @@ bool QMakeParser::read(ProFile *pro, const QString &in, int line, SubGrammar gra
lineEnd: lineEnd:
if (lineCont) { if (lineCont) {
if (quote) { if (quote) {
putSpace = true; *ptr++ = ' ';
} else { } else {
FLUSH_LITERAL(); FLUSH_LITERAL();
needSep = TokNewStr; needSep = TokNewStr;