Re: Ticket 162, was: Ambiguity in ABNF for comment

Brian Smith wrote:
> Julian Reschke wrote:
>> Proposed patch; change ctext to say:
>>
>>        ctext          = *( OWS / %x21-27 / %x2A-5B / %x5D-7E / obs-
>> text )
> 
> It makes perfect sense, once one realizes that x28, x29, and x5C are "(",
> ")", and "\" respectively. 

Indeed.

> Nitpick: The Kleene star is unnecessary and inconsistent with obs-text. This
> is enough:
> 
>     ctext = OWS / %x21-27 / %x2A-5B / %x5D-7E / obs-text

Correct.

So how about:

      quoted-string  = DQUOTE *( qdtext / quoted-pair ) DQUOTE
      qdtext         = OWS / %x21 / %x23-5B / %x5D-7E / obs-text
                     ; OWS / VCHAR except DQUOTE / obs-text
      obs-text       = %x80-FF

and

      comment        = "(" *( ctext / quoted-pair / comment ) ")"
      ctext          = OWS / %x21-27 / %x2A-5B / %x5D-7E / obs-text
                     ; OWS / VCHAR except "(", ")" and "\" / obs-text

then? See also 
<http://trac.tools.ietf.org/wg/httpbis/trac/attachment/ticket/162/162.diff>.

Furthermore, am I missing something, or can't we also remove "comment" 
from the right hand side of the "comment" production?


BR, Julian

Received on Saturday, 2 May 2009 09:32:52 UTC