[PATCH] D89765: [LibTooling][Clang-cast] A Clang LibTool to convert C-style casts to C++ style casts and more.

[Ray Zhang via Phabricator via cfe-commits]

oneraynyday updated this revision to Diff 299274.
oneraynyday added a comment.

Don't use C++17 features as clang currently uses C++14.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D89765/new/

https://reviews.llvm.org/D89765

Files:
  clang-tools-extra/clang-cast/Matcher.h


Index: clang-tools-extra/clang-cast/Matcher.h
===================================================================
--- clang-tools-extra/clang-cast/Matcher.h
+++ clang-tools-extra/clang-cast/Matcher.h
@@ -169,7 +169,9 @@
 // TODO: Is this okay to do?
 Matcher::~Matcher() {
   if (PublishSummary) {
-    for (auto const &[CXXCastKind, Freq] : Statistics) {
+    for (const auto &Pair : Statistics) {
+      const auto &CXXCastKind = Pair.first;
+      const auto &Freq = Pair.second;
       if (!Freq)
         continue;
       llvm::errs() << "The type " << cppCastToString(CXXCastKind)



Index: clang-tools-extra/clang-cast/Matcher.h
===================================================================
--- clang-tools-extra/clang-cast/Matcher.h
+++ clang-tools-extra/clang-cast/Matcher.h
@@ -169,7 +169,9 @@
 // TODO: Is this okay to do?
 Matcher::~Matcher() {
   if (PublishSummary) {
-    for (auto const &[CXXCastKind, Freq] : Statistics) {
+    for (const auto &Pair : Statistics) {
+      const auto &CXXCastKind = Pair.first;
+      const auto &Freq = Pair.second;
       if (!Freq)
         continue;
       llvm::errs() << "The type " << cppCastToString(CXXCastKind)

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