Re: Request to review sieve URI scheme

Hi Bjoern,

Bjoern Hoehrmann wrote:

>* Alexey Melnikov wrote:
>  
>
>>I am finishing draft-ietf-sieve-managesieve-02.txt which defines the 
>>sieve: URI scheme. So I would like to ask people to review section 3 of 
>>the draft.
>>    
>>
>You have a reference [IMAP4] but it's not part of the References in the
>document. I assume this should be [IMAP4rev1].
>  
>
Thanks for spotting that. Fixed.

>You should say upfront that `authority`, `unreserved`, etc. are defined
>in STD066 and included by reference. Right now you are using authority
>without defining it, and repeat the reference multiple times.
>
Yes. I had a note at the top of the ABNF, but I guess it wasn't clear.

>There is not much point in having the sub-delims-sh and uchar rules in
>the grammar, you could just use
>
>  ochar = <pchar but not ";", "&", or "=">
>
>or the equivalent (I assume)