[PATCH] D52727: [clang-tidy] White List Option for performance-unnecessary-value-param, performance-unnecessary-copy-initialization and performance-for-range-copy

[Aaron Ballman via Phabricator via cfe-commits]

aaron.ballman added inline comments.


================
Comment at: clang-tidy/performance/ForRangeCopyCheck.h:43
   const bool WarnOnAllAutoCopies;
+  const std::vector<std::string> WhiteListTypes;
 };
----------------
Nit: can you name it `AllowedTypes` rather than `WhiteListTypes`? Use similar 
nomenclature ("allow" instead of "white") elsewhere as well.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D52727



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