Incorrect "wrong syntax" error reported when link to this version has trailing spaces

[tidoust]

If the spec to test contains trailing spaces in the href attribute of the "This version" and other "Previous version" links, Specberus reports errors such as:

• Wrong syntax for This Version link.
• error Short names differ between This and Latest Versions.
• error Short names differ between This and Previous Versions.

Example of markup that triggers the issue:

<dl>
 <dt>This version:</dt>
 <dd><a href="http://www.w3.org/TR/2015/WD-presentation-api-20150610/ ">http://www.w3.org/TR/2015/WD-presentation-api-20150610/ </a>
 </dd>
 <dt>Latest published version:</dt>
 <dd><a href="http://www.w3.org/TR/presentation-api/ ">http://www.w3.org/TR/presentation-api/</a></dd>
 ...
</dl>

I don't have an example of an online spec because I'm going to update the version that I was testing. The thing is: Anolis generates this space, which is technically correct although a tad annoying perhaps, so it's not that easy to fix in the generated spec.

In the example, removing the final space before the closing double-quote in the href attribute fixes the issue. I suspect the code that would need to be tweaked is in lib/rules/headers/dl.js:

https://github.com/w3c/specberus/blob/master/lib/rules/headers/dl.js#L87

In particular, adding a call to trim() should do the trick but note it would have to be done for the latest link and the previous link as well:

var matches = ($linkThis.attr("href") || "").trim().match(new RegExp(vThisRex))

[tidoust]

Hmm, actually, I was wrong, Anolis only produces the space if there is a similar space in the source text. In practice, Anolis generates the links based from text in the source spec that is in-between <!--begin-link--> and <!--end-link--> comments, and will just preserve the spaces that it found here. Anyway, the issue exists on Specberus, but I can easily fix it on my side, so not a priority at all.