[PATCH] D32346: [clang-tidy] New readability check for strlen argument

[Stephan Bergmann via Phabricator via cfe-commits]

sberg added inline comments.


================
Comment at: test/clang-tidy/readability-strlen-argument.cpp:11
+  X = strlen(Str + 10);
+  // CHECK-FIXES: {{^}}  X = (strlen(Str) - 10);{{$}}
+
----------------
but if any of the first ten chars in Str can be NUL, then the change is bogus?  
(I guess I fail to see the reason for this check)


Repository:
  rL LLVM

https://reviews.llvm.org/D32346



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