[PATCH] D28667: [clang-tidy] Don't modernize-raw-string-literal if replacement is longer.

AndrĂ¡s Leitereg via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Jan 15 23:41:58 PST 2017


leanil added inline comments.


================
Comment at: test/clang-tidy/modernize-raw-string-literal.cpp:94
 
+char const *const Concatenated("\"foo\""
+                               "\"bar\"");
----------------
alexfh wrote:
> Does this test fail without the patch? Also, should there be a test covering the case where no replacement is made (and the current code issues a warning)?
No, it doesn't. This is to check that MeasureTokenLength works as I expect. It wouldn't warn if it measured the two parts separately.
And I left the check here for those who may have the same concerns, although it's indeed not the core functionality I test here. Should I remove it?


https://reviews.llvm.org/D28667





More information about the cfe-commits mailing list