[PATCH] D16152: [clang-tidy] Add check performance-faster-string-find

[Aaron Ballman via cfe-commits]

aaron.ballman added inline comments.

================
Comment at: clang-tidy/performance/FasterStringFindCheck.cpp:25
@@ +24,3 @@
+  SmallVector<StringRef, 4> Classes;
+  Option.split(Classes, ",");
+  return std::vector<std::string>(Classes.begin(), Classes.end());
----------------
alexfh wrote:
> aaron.ballman wrote:
> > It might be nice for this to be more tolerant of whitespace around the commas.
> I don't see much value in this, since it's not someone will change frequently (I guess, once per codebase on average).
> I don't see much value in this, since it's not someone will change frequently (I guess, once per codebase on average).

Anything that's user-facing should be fault tolerant and sanitized for a better user experience.


http://reviews.llvm.org/D16152