The new version is looking pretty good to me!
My main concern still is that specification of whitespace handling is still kinda confusing/underspecified. The text says "all whitespace is optional", but the grammar says that it's mandatory in some cases (e.g. url-marker, still not sure why -- I'd understand if it were mandatory before the ";" since ";" is a valid character in URLs, but it says it's mandatory afterward?), and the grammar is still wrong about whitespace in some cases (e.g. it says ">= 1.0" is an illegal versionspec).
I guess the two options are either to go through carefully sprinkling *WSP's about at all the appropriate places, or else to tackle things more systematically by adding a lexer layer...
Also, unrelated: do you want to import the text for PEP 426 about the requirements for hashes in URLs?
On Sun, Nov 8, 2015 at 1:48 PM, Robert Collins firstname.lastname@example.org wrote:
Bah, just remembered to trim the marker operators too. Doing so now, I won't include a diff for that.
-- Nathaniel J. Smith -- http://vorpus.org