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



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

Reply via email to