[Bug 12657] Regex for durationLexicalRep in 3.3.7.2 Lexical Mapping

http://www.w3.org/Bugs/Public/show_bug.cgi?id=12657

Dave Peterson <davep@iit.edu> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |davep@iit.edu

--- Comment #1 from Dave Peterson <davep@iit.edu> 2011-05-14 18:55:58 UTC ---
(In reply to comment #0)

> > at least one field occurs.

> The regex given thereafter (on six lines) does not seem to satisfy the second
> requirement. It seems that five question marks need to be removed from the
> regex. 

You have indeed found an oops! in the published regex, but the fix is not as
drastic as you suggest.

Each lin in the regex is intended to make one of the fields required, the first
field in the regex.  It presumes all preceding fields are missing (or a
preceding line would be matched), and it makes all following fields optional. 
Thus, one of the six lines should match starting with the first actually
present field.  Each first-in-the-line field of the regex should be required,
i.e., not followed by a question mark.  There are two lines in which a spurious
question mark on the first field is present and must be removed:  the line
beginning '|([0-9]+D)?' should have that question mark removed; similarly for
the line beginning '|([0-9]+M)?'.  @%#%& cut-and-paste errors.

Sigh.  You can't imagine how many times that regex has been checked.  :-(

-- 
Configure bugmail: http://www.w3.org/Bugs/Public/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.

Received on Saturday, 14 May 2011 18:56:01 UTC