Re: #173: CR and LF in chunk extension values

Mark Nottingham wrote:
> ...
>> 3)
>>
>> It seems to me that the purpose of quoted-text is to allow any 
>> character in qdtext, plus DQUOTE and "\", which would make it
>>
>> quoted-text = qdtext / DQUOTE / "\"
>>
>> While we're at it, we probably should rename it to quoted-char, and 
>> also add a short statement what the semantics of a quoted-pair is.
> 
> 
> I had to read that a few times, but I think I agree. However, 
> "quoted-char" may be confusing, as it's very similar to "quoted-pair".

And yes,

	qdtext / DQUOTE / "\"

is the same as

	WSP / VCHAR / obs-text

...but I think the former is more clear in that it adds DQUOTE and "\".

But.

quoted-pair is also used in comments. Are we ok with restricting the set 
here as well? And, if yes, shouldn't we then also adjust the allowed set 
for non-quoted characters in comments?

Currently it reads 
(<http://greenbytes.de/tech/webdav/draft-ietf-httpbis-p1-messaging-latest.html#rfc.section.3.2>):

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

To make it consistent with quoted-string it would need to change to:

   ctext          = BWS / %x21-27 / %x2A-5B / %x5D-7E / obs-text
                  ; BWS / <VCHAR except "(", ")", and "\"> / obs-text

Feedback appreciated,

Julian

Received on Tuesday, 25 August 2009 07:11:56 UTC