Use range-based for loops for better readability. No functional changes intended.

Reviewers: chandlerc

Reviewed By: chandlerc

CC: cfe-commits

Differential Revision: http://llvm-reviews.chandlerc.com/D2979

llvm-svn: 203101

1 files changed, 7 insertions, 10 deletions

diff --git a/clang-tools-extra/clang-tidy/ClangTidyModule.cpp b/clang-tools-extra/clang-tidy/ClangTidyModule.cpp
index 87ab2be..8a6842b 100644
--- a/clang-tools-extra/clang-tidy/ClangTidyModule.cpp
+++ b/clang-tools-extra/clang-tidy/ClangTidyModule.cpp
@@ -17,24 +17,21 @@ namespace clang {
 namespace tidy {
 
 ClangTidyCheckFactories::~ClangTidyCheckFactories() {
-  for (FactoryMap::iterator I = Factories.begin(), E = Factories.end(); I != E;
-       ++I) {
-    delete I->second;
-  }
+  for (const auto &Factory : Factories)
+    delete Factory.second;
 }
+
 void ClangTidyCheckFactories::addCheckFactory(StringRef Name,
                                               CheckFactoryBase *Factory) {
-
   Factories[Name] = Factory;
 }
 
 void ClangTidyCheckFactories::createChecks(
     ChecksFilter &Filter, SmallVectorImpl<ClangTidyCheck *> &Checks) {
-  for (FactoryMap::iterator I = Factories.begin(), E = Factories.end(); I != E;
-       ++I) {
-    if (Filter.IsCheckEnabled(I->first)) {
-      ClangTidyCheck *Check = I->second->createCheck();
-      Check->setName(I->first);
+  for (const auto &Factory : Factories) {
+    if (Filter.IsCheckEnabled(Factory.first)) {
+      ClangTidyCheck *Check = Factory.second->createCheck();
+      Check->setName(Factory.first);
       Checks.push_back(Check);
     }
   }