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

From: daniel_james_at_[hidden]
Date: 2008-06-27 12:52:27

Author: danieljames
Date: 2008-06-27 12:52:27 EDT (Fri, 27 Jun 2008)
New Revision: 46779
URL: http://svn.boost.org/trac/boost/changeset/46779

Log:
Better message when there's not hostname. And add a comment explaining the
reasoning behind this rule.

Text files modified: 
   trunk/tools/inspect/link_check.cpp |     5 ++++-                                   
   1 files changed, 4 insertions(+), 1 deletions(-)

Modified: trunk/tools/inspect/link_check.cpp
==============================================================================
--- trunk/tools/inspect/link_check.cpp	(original)
+++ trunk/tools/inspect/link_check.cpp	2008-06-27 12:52:27 EDT (Fri, 27 Jun 2008)
@@ -150,10 +150,13 @@
       // Protocol checks
       if(scheme_matched) {
         if(scheme == "http" || scheme == "https") {
+          // All http links should have a hostname. Generally if they don't
+          // it's by mistake. If they shouldn't, then a protocol isn't
+          // required.
           if(!authority_matched) {
             if(!no_link_errors) {
               ++m_invalid_errors;
-              error( library_name, source_path, string(name()) + " http protocol without hostname: " + url );
+              error( library_name, source_path, string(name()) + " no hostname: " + url );
             }
           }