alexfh added inline comments.
================
Comment at: clang-tidy/modernize/DeprecatedHeadersCheck.cpp:83
@@ +82,3 @@
+ for (const auto &Key : DeleteHeaders) {
+ DeleteHeadersSet.insert(Key);
+ }
----------------
No danger of dangling references here, since `StringMap` copies keys. See
`StringMap` comments and `StringMapEntry::Create` implementation.
So there's no need to store another copy of this data.
Kirill, please revert the latest change in this constructor.
https://reviews.llvm.org/D17990
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
