[inspect][1.35] Link check fixes

Hi, Some of the link errors in the inspect report are incorrect. I checked some fixes into trunk: http://svn.boost.org/trac/boost/changeset/43750 http://svn.boost.org/trac/boost/changeset/43752 The first one causes it to check links in single quotes - this fixes a couple of incorrect unlinked file reports in iostream (although 'libs/iostreams/doc/concepts/multi-character.html' is unlinked). The second changeset decodes urls with percent encoded characters and encoded ampersands - this fixes about 200 incorrect link errors in the doc directory (but there are a few left). There's no real need to add these changes to release at this late stage, just note that some of the link errors are incorrect. Daniel

Daniel James wrote:
Hi,
Some of the link errors in the inspect report are incorrect. I checked some fixes into trunk:
http://svn.boost.org/trac/boost/changeset/43750 http://svn.boost.org/trac/boost/changeset/43752
The first one causes it to check links in single quotes - this fixes a couple of incorrect unlinked file reports in iostream (although 'libs/iostreams/doc/concepts/multi-character.html' is unlinked). The second changeset decodes urls with percent encoded characters and encoded ampersands - this fixes about 200 incorrect link errors in the doc directory (but there are a few left).
Wow! Thanks! I wasn't even aware that some of the link errors were phony.
There's no real need to add these changes to release at this late stage, just note that some of the link errors are incorrect.
I rebuilt and ran with your changes. The number of problems reported dropped from 1100 to 862. Unless you object, I'd like to merge these changes into the release branch. Thanks, --Beman

On 21/03/2008, Beman Dawes <bdawes@acm.org> wrote:
Wow! Thanks! I wasn't even aware that some of the link errors were phony.
I had a quick look into this and most of these errors are recent. Boostbook has started including type names in some reference documentation filenames. So you get spaces for types like 'long long'. Which is a little annoying but IMO an issue for the release. Bit I'll try to change it for the next release (http://svn.boost.org/trac/boost/ticket/1705).
Unless you object, I'd like to merge these changes into the release branch.
I've merged them now. Daniel
participants (2)
-
Beman Dawes
-
Daniel James