[PATCH] Support for CR LF newlines.

Alexander Kornienko alexfh at google.com
Wed Sep 11 03:09:17 PDT 2013



================
Comment at: unittests/Format/FormatTest.cpp:119
@@ -118,3 +118,3 @@
   EXPECT_EQ("int i;", format("  int i;"));
-  EXPECT_EQ("\nint i;", format(" \n\t \r  int i;"));
+  EXPECT_EQ("\nint i;", format(" \n\t \v \f  int i;"));
   EXPECT_EQ("int i;\nint j;", format("    int i; int j;"));
----------------
Daniel Jasper wrote:
> What happens with this test if you leave the \r?
The result gets \r\n newline, which is not what we're trying to check in this test.


http://llvm-reviews.chandlerc.com/D1643



More information about the cfe-commits mailing list