[Boost-commit] svn:boost r63218 - trunk/tools/inspect

Subject: [Boost-commit] svn:boost r63218 - trunk/tools/inspect
From: steven_at_[hidden]
Date: 2010-06-21 23:30:13

Author: steven_watanabe
Date: 2010-06-21 23:30:12 EDT (Mon, 21 Jun 2010)
New Revision: 63218
URL: http://svn.boost.org/trac/boost/changeset/63218

Log:
Adjust heading in the inspection report.  Fixes #4134.
Text files modified: 
   trunk/tools/inspect/apple_macro_check.cpp |     7 ++++++-                                 
   trunk/tools/inspect/apple_macro_check.hpp |     2 +-                                      
   trunk/tools/inspect/link_check.cpp        |     2 +-                                      
   3 files changed, 8 insertions(+), 3 deletions(-)

Modified: trunk/tools/inspect/apple_macro_check.cpp
==============================================================================
--- trunk/tools/inspect/apple_macro_check.cpp	(original)
+++ trunk/tools/inspect/apple_macro_check.cpp	2010-06-21 23:30:12 EDT (Mon, 21 Jun 2010)
@@ -61,6 +61,8 @@
 
       boost::sregex_iterator cur(contents.begin(), contents.end(), apple_macro_regex), end;
 
+      long errors = 0;
+
       for( ; cur != end; ++cur /*, ++m_files_with_errors*/ )
       {
 
@@ -79,12 +81,15 @@
               }
           }
 
-          ++m_files_with_errors;
+          ++errors;
           error( library_name, full_path, 
             "Apple macro clash: " + std::string((*cur)[0].first, (*cur)[0].second-1),
             line_number );
         }
       }
+      if(errors > 0) {
+        ++m_files_with_errors;
+      }
     }
   } // namespace inspect
 } // namespace boost

Modified: trunk/tools/inspect/apple_macro_check.hpp
==============================================================================
--- trunk/tools/inspect/apple_macro_check.hpp	(original)
+++ trunk/tools/inspect/apple_macro_check.hpp	2010-06-21 23:30:12 EDT (Mon, 21 Jun 2010)
@@ -31,7 +31,7 @@
         const std::string & contents );
 
       virtual ~apple_macro_check()
-        { std::cout << "  " << m_files_with_errors << " files with apple macros" << line_break(); }
+        { std::cout << "  " << m_files_with_errors << " files with Apple macros" << line_break(); }
     };
   }
 }

Modified: trunk/tools/inspect/link_check.cpp
==============================================================================
--- trunk/tools/inspect/link_check.cpp	(original)
+++ trunk/tools/inspect/link_check.cpp	2010-06-21 23:30:12 EDT (Mon, 21 Jun 2010)
@@ -368,7 +368,7 @@
             // since some browsers do a case-sensitive match.
             && bookmarks.find(decode_percents(fragment)) == bookmarks.end() )
           {
-            ++m_bookmark_errors;
+            ++m_broken_errors;
             int ln = std::count( contents_begin, url_start, '\n' ) + 1;
             error( library_name, source_path, "Unknown bookmark: " + decoded_url, ln );
           }