Re: [PATCH] D12933: Add a test to modernize-loop-convert.

2015-09-17 Thread Angel Garcia via cfe-commits
angelgarcia updated this revision to Diff 34993. angelgarcia added a comment. Done. http://reviews.llvm.org/D12933 Files: test/clang-tidy/modernize-loop-convert-extra.cpp Index: test/clang-tidy/modernize-loop-convert-extra.cpp =

Re: [PATCH] D12933: Add a test to modernize-loop-convert.

2015-09-17 Thread Alexander Kornienko via cfe-commits
alexfh accepted this revision. alexfh added a comment. This revision is now accepted and ready to land. LG with a comment. Comment at: test/clang-tidy/modernize-loop-convert-extra.cpp:705 @@ -704,16 +704,3 @@ - // FIXME: Right now, clang-tidy does not allow to make insertions

[PATCH] D12933: Add a test to modernize-loop-convert.

2015-09-17 Thread Angel Garcia via cfe-commits
angelgarcia created this revision. angelgarcia added a reviewer: alexfh. angelgarcia added subscribers: klimek, cfe-commits. Add the test about replacements in several arguments of the same macro call, now that the problem has been fixed. http://reviews.llvm.org/D12933 Files: test/clang-tidy/