Michael Haardt wrote:
The update proposal is like this:
NEW:
blank = WSP / CRLF
encoded-arb-octets = "${hex:" hex-pair-seq "}"
hex-pair-seq = *blank hex-pair *(1*blank hex-pair) *blank
^^^^^^ ^^^^^^^ ^^^^^^
The white space looks fine, but you missed defining hex-pair.
No, it just stays as it is defined now. That is why I haven't listed it.
NEW:
encoded-unicode-char = "${unicode:" unicode-hex-seq "}"
unicode-hex-seq = *blank unicode-hex *(1*blank unicode-hex) *blank
^^^^^^ ^^^^^^^ ^^^^^^
unicode-hex = 1*6HEXDIG
You did define unicode-hex here, but unchanged, not as 1*HEXDIG.
Ah, sorry about that. Yes, I meant "1*HEXDIG".
How
about that?
blank = WSP / CRLF
encoded-arb-octets = "${hex:" *blank hex-seq *blank "}"
encoded-unicode-chars = "${unicode:" *blank hex-seq *blank "}"
hex-seq = hex *(1*blank hex)
hex = 1*HEXDIG
IMHO, the leading and trailing white space is not part of the sequence.
If you think different, then put it back into hex-seq.
I think they are part of the hex-seq, but this is not something to argue
about.
If we omit the length restriction, we may as well merge the sequences into one
production.
I think we've only agreed to remove length restriction on "${unicode:".
BTW, are "${hex:"/"£{unicode:" prefixes case insensitive? I say yes.