In html::EscapeText, instead of going through the rewriter with
a SourceLocation to get a RewriteBuffer, poke the RewriteBuffer
with an offset directly. THis is no faster, but results in
cleaner code.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@49774 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/lib/Rewrite/HTMLRewrite.cpp b/lib/Rewrite/HTMLRewrite.cpp
index fc1627a..b4dbb44 100644
--- a/lib/Rewrite/HTMLRewrite.cpp
+++ b/lib/Rewrite/HTMLRewrite.cpp
@@ -30,49 +30,38 @@
assert (C <= FileEnd);
+ RewriteBuffer &RB = R.getEditBuffer(FileID);
+
for (unsigned FilePos = 0; C != FileEnd ; ++C, ++FilePos) {
switch (*C) {
- default: break;
-
- case ' ':
- if (EscapeSpaces) {
- SourceLocation Loc = SourceLocation::getFileLoc(FileID, FilePos);
- R.ReplaceText(Loc, 1, " ", 6);
- }
- break;
+ default: break;
+
+ case ' ':
+ if (EscapeSpaces)
+ RB.ReplaceText(FilePos, 1, " ", 6);
+ break;
- case '\t': {
- if (!ReplaceTabs)
- break;
-
- SourceLocation Loc = SourceLocation::getFileLoc(FileID, FilePos);
-
- if (EscapeSpaces)
- R.ReplaceText(Loc, 1, " ", 6*4);
- else
- R.ReplaceText(Loc, 1, " ", 4);
-
+ case '\t':
+ if (!ReplaceTabs)
break;
- }
-
- case '<': {
- SourceLocation Loc = SourceLocation::getFileLoc(FileID, FilePos);
- R.ReplaceText(Loc, 1, "<", 4);
- break;
- }
-
- case '>': {
- SourceLocation Loc = SourceLocation::getFileLoc(FileID, FilePos);
- R.ReplaceText(Loc, 1, ">", 4);
- break;
- }
-
- case '&': {
- SourceLocation Loc = SourceLocation::getFileLoc(FileID, FilePos);
- R.ReplaceText(Loc, 1, "&", 5);
- break;
- }
+ if (EscapeSpaces)
+ RB.ReplaceText(FilePos, 1, " ", 6*4);
+ else
+ RB.ReplaceText(FilePos, 1, " ", 4);
+ break;
+
+ case '<':
+ RB.ReplaceText(FilePos, 1, "<", 4);
+ break;
+
+ case '>':
+ RB.ReplaceText(FilePos, 1, ">", 4);
+ break;
+
+ case '&':
+ RB.ReplaceText(FilePos, 1, "&", 5);
+ break;
}
}
}
@@ -98,7 +87,8 @@
case '\t':
if (ReplaceTabs)
- for (unsigned i = 0; i < 4; ++i) os << " ";
+ for (unsigned i = 0; i < 4; ++i)
+ os << " ";
else os << c;
break;