[PATCH] D13052: Add NamingStyle option to modernize-loop-convert.

Alexander Kornienko via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 24 07:40:01 PDT 2015

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

Looks good with a comment. Thank you!

Comment at: test/clang-tidy/modernize-loop-convert-uppercase.cpp:48
@@ +47,3 @@
+  // CHECK-FIXES: for (auto & NUMS_ELEM : NUMS)
+  // CHECK-FIXES-NEXT: printf("%d\n", NUMS_ELEM + NUM + ELEM + NUMS_I);
Please add a test for GIVE_ME_NAME_{0,1}. Same for other styles.


More information about the cfe-commits mailing list