It turned out just to be an optimization. It does less work, not
more. Do you have a suggestion of what would be a good test case for
this? Testing this has more to do with the browser behavior than the
emitted HTML.
On Mar 18, 2009, at 1:04 PM, Daniel Dunbar wrote:
Test case for this isn't out of the question?
- Daniel
On Wed, Mar 18, 2009 at 11:47 AM, Ted Kremenek <[email protected]>
wrote:
Author: kremenek
Date: Wed Mar 18 13:47:46 2009
New Revision: 67221
URL: http://llvm.org/viewvc/llvm-project?rev=67221&view=rev
Log:
HTML rewriter: Don't replace tabs with spaces by default.
Modified:
cfe/trunk/include/clang/Rewrite/HTMLRewrite.h
Modified: cfe/trunk/include/clang/Rewrite/HTMLRewrite.h
URL:
http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Rewrite/HTMLRewrite.h?rev=67221&r1=67220&r2=67221&view=diff
=
=
=
=
=
=
=
=
======================================================================
--- cfe/trunk/include/clang/Rewrite/HTMLRewrite.h (original)
+++ cfe/trunk/include/clang/Rewrite/HTMLRewrite.h Wed Mar 18
13:47:46 2009
@@ -52,14 +52,14 @@
/// EscapeText - HTMLize a specified file so that special
characters are
/// are translated so that they are not interpreted as HTML tags.
void EscapeText(Rewriter& R, FileID FID,
- bool EscapeSpaces = false, bool ReplacesTabs =
true);
+ bool EscapeSpaces = false, bool ReplaceTabs =
false);
/// EscapeText - HTMLized the provided string so that special
characters
/// in 's' are not interpreted as HTML tags. Unlike the version of
/// EscapeText that rewrites a file, this version by default
replaces tabs
/// with spaces.
std::string EscapeText(const std::string& s,
- bool EscapeSpaces = false, bool
ReplaceTabs = true);
+ bool EscapeSpaces = false, bool
ReplaceTabs = false);
void AddLineNumbers(Rewriter& R, FileID FID);
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
