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

[Alexander Kornienko via cfe-commits]

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.


http://reviews.llvm.org/D13052