[PATCH] D17990: [clang-tidy] minor improvements in modernise-deprecated-headers check

Alexander Kornienko via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 19 07:31:56 PDT 2016


alexfh requested changes to this revision.
This revision now requires changes to proceed.

================
Comment at: clang-tidy/modernize/DeprecatedHeadersCheck.cpp:39
@@ -37,2 +38,3 @@
   llvm::StringMap<std::string> CStyledHeaderToCxx;
+  std::set<std::string> DeleteHeader;
 };
----------------
nit: DeleteHeaders

================
Comment at: clang-tidy/modernize/DeprecatedHeadersCheck.cpp:39
@@ -37,2 +38,3 @@
   llvm::StringMap<std::string> CStyledHeaderToCxx;
+  std::set<std::string> DeleteHeader;
 };
----------------
alexfh wrote:
> nit: DeleteHeaders
Use `llvm::StringSet`


https://reviews.llvm.org/D17990





More information about the cfe-commits mailing list