alexfh accepted this revision.
alexfh added a comment.
This revision is now accepted and ready to land.

Looks good with a comment.


================
Comment at: test/clang-tidy/modernize-loop-convert-extra.cpp:636
@@ +635,3 @@
+       SE = container.end(); SI != SE; ++SI) {
+  }
+}
----------------
Please add a test to ensure that loops over non-dependent container types in 
templates are still converted.


http://reviews.llvm.org/D12321



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

Reply via email to