diff options
author | Dimitri van Heesch <dimitri@stack.nl> | 2014-08-03 16:19:24 (GMT) |
---|---|---|
committer | Dimitri van Heesch <dimitri@stack.nl> | 2014-08-03 16:19:24 (GMT) |
commit | 1be08948215fa665cc7e6779422ebf81a982de50 (patch) | |
tree | 03ee44d9d89bf8bbf58182c4cb93cb40fc8f2632 /src | |
parent | 8eca4c66338b96d1d7d7ef68ddf34b2274729d5b (diff) | |
parent | d4601735b582b903f1ccb144f59b2030a7797b05 (diff) | |
download | Doxygen-1be08948215fa665cc7e6779422ebf81a982de50.zip Doxygen-1be08948215fa665cc7e6779422ebf81a982de50.tar.gz Doxygen-1be08948215fa665cc7e6779422ebf81a982de50.tar.bz2 |
Merge pull request #206 from groleo/master
removeRedundantWhiteSpace micro-optimization
Diffstat (limited to 'src')
-rw-r--r-- | src/util.cpp | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/src/util.cpp b/src/util.cpp index 7fff1b9..be68f08 100644 --- a/src/util.cpp +++ b/src/util.cpp @@ -1667,10 +1667,11 @@ QCString removeRedundantWhiteSpace(const QCString &s) uint l=s.length(); uint csp=0; uint vsp=0; + char c; for (i=0;i<l;i++) { nextChar: - char c=s.at(i); + c=s.at(i); // search for "const" if (csp<6 && c==constScope[csp] && // character matches substring "const" @@ -1705,7 +1706,7 @@ nextChar: if (cc=='\\') // escaped character { growBuf.addChar(s.at(i+1)); - i+=2; + i+=2; } else if (cc=='"') // end of string { i++; goto nextChar; } @@ -1737,14 +1738,16 @@ nextChar: growBuf.addChar(','); growBuf.addChar(' '); } - else if (i>0 && - ((isId(s.at(i)) && s.at(i-1)==')') || - (s.at(i)=='\'' && s.at(i-1)==' ') + else if (i>0 && + ( + (s.at(i-1)==')' && isId(c)) + || + (c=='\'' && s.at(i-1)==' ') ) ) { growBuf.addChar(' '); - growBuf.addChar(s.at(i)); + growBuf.addChar(c); } else if (c=='t' && csp==5 /*&& (i<5 || !isId(s.at(i-5)))*/ && !(isId(s.at(i+1)) /*|| s.at(i+1)==' '*/ || |