LegalizeAdulthood marked an inline comment as done.

================
Comment at: clang-tidy/modernize/RawStringLiteralCheck.cpp:79
@@ +78,3 @@
+  if (Text.find('R') < Text.find('"'))
+    return false;
+
----------------
Take a look at http://en.cppreference.com/w/cpp/language/string_literal

There can be a prefix before the R, it's not always the first character.

================
Comment at: clang-tidy/modernize/RawStringLiteralCheck.cpp:87
@@ +86,3 @@
+  return containsEscapes(Text, R"lit('\"?x01)lit");
+}
+
----------------
Without R"()", the \ and " have to be escaped.  Running this check on this code 
would turn this into a raw string literal.

================
Comment at: docs/clang-tidy/checks/modernize-raw-string-literal.rst:46
@@ +45,3 @@
+editor.  Trailing whitespace is likely to be stripped by editors and other
+tools, changing the meaning of the literal.
+
----------------
It's a habit my fingers learned when they taught me to type on typewriters in 
high school.


http://reviews.llvm.org/D16529



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to