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 );
}
}