[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