Converted a load of spelt out iterators to auto

include/internal/catch_run_context.cpp

@@ -277,7 +277,7 @@ namespace Catch {
     void RunContext::handleUnfinishedSections() {
         // If sections ended prematurely due to an exception we stored their
         // infos here so we can tear them down outside the unwind process.
-        for (std::vector<SectionEndInfo>::const_reverse_iterator it = m_unfinishedSections.rbegin(),
+        for (auto it = m_unfinishedSections.rbegin(),
              itEnd = m_unfinishedSections.rend();
              it != itEnd;
              ++it)

include/internal/catch_tag_alias_registry.cpp

@@ -16,7 +16,7 @@ namespace Catch {
     TagAliasRegistry::~TagAliasRegistry() {}
 
     Option<TagAlias> TagAliasRegistry::find( std::string const& alias ) const {
-        std::map<std::string, TagAlias>::const_iterator it = m_registry.find( alias );
+        auto it = m_registry.find( alias );
         if( it != m_registry.end() )
             return it->second;
         else

include/internal/catch_test_case_tracker.hpp

@@ -167,7 +167,7 @@ namespace TestCaseTracking {
         }
 
         virtual ITrackerPtr findChild( NameAndLocation const& nameAndLocation ) override {
-            Children::const_iterator it = std::find_if( m_children.begin(), m_children.end(), TrackerHasName( nameAndLocation ) );
+            auto it = std::find_if( m_children.begin(), m_children.end(), TrackerHasName( nameAndLocation ) );
             return( it != m_children.end() )
                 ? *it
                 : nullptr;

include/reporters/catch_reporter_bases.cpp

@@ -120,7 +120,7 @@ namespace Catch {
             node = m_rootSection;
         } else {
             SectionNode& parentNode = *m_sectionStack.back();
-            SectionNode::ChildSections::const_iterator it =
+            auto it =
                 std::find_if(parentNode.childSections.begin(),
                              parentNode.childSections.end(),
                              BySectionInfo(sectionInfo));

include/reporters/catch_reporter_compact.cpp

@@ -217,7 +217,7 @@ namespace Catch {
                 if ( itMessage == messages.end() )
                     return;
 
-                // using messages.end() directly yields compilation error:
+                // using messages.end() directly yields (or auto) compilation error:
                 std::vector<MessageInfo>::const_iterator itEnd = messages.end();
                 const std::size_t N = static_cast<std::size_t>( std::distance( itMessage, itEnd ) );
 

include/reporters/catch_reporter_console.cpp

@@ -274,7 +274,7 @@ namespace Catch {
             if( m_sectionStack.size() > 1 ) {
                 Colour colourGuard( Colour::Headers );
 
-                std::vector<SectionInfo>::const_iterator
+                auto
                     it = m_sectionStack.begin()+1, // Skip first section (test case)
                     itEnd = m_sectionStack.end();
                 for( ; it != itEnd; ++it )

include/reporters/catch_reporter_tap.hpp

@@ -205,7 +205,7 @@ namespace Catch {
                     return;
                 }
 
-                // using messages.end() directly yields compilation error:
+                // using messages.end() directly (or auto) yields compilation error:
                 std::vector<MessageInfo>::const_iterator itEnd = messages.end();
                 const std::size_t N = static_cast<std::size_t>( std::distance( itMessage, itEnd ) );