Re: [Extra] Comments on draft-ietf-extra-sieve-special-use-03

[Stephan Bosch <[e-mail]>]

Hi Alexey,


Op 28-10-2018 om 16:07 schreef Alexey Melnikov:
> Hi all,
> I've just tried to implement this Sieve extension and I have a couple of
> comments:
>
> 3.  Test "specialuse_exists"
>
>     Usage:  specialuse_exists [<mailbox: string>]
>                               <special-use-flags: string-list>
>
> This defines an optional argument. According to the base Sieve RFC 5228,
> optional arguments must be tagged:
>
>    2.6.1.  Positional Arguments
>
>     Positional arguments are given to a command that discerns their
>     meaning based on their order.  When a command takes positional
>     arguments, all positional arguments must be supplied and must be in
>     the order prescribed.
>
>   and
>
>    2.6.3.  Optional Arguments
>
>     Optional arguments are exactly like tagged arguments except that they
>     may be left out, in which case a default value is implied.  Because
>     optional arguments tend to result in shorter scripts, they have been
>     used far more than tagged arguments.
>
> So I don't think you can have "mailbox" being positional without a tag!

Then what about the entirety of the "imap4flags" extension? Lots of 
tag-less optional arguments can be found there. I am not against having 
some ":mailbox" tag here, but the current syntax does make things more 
concise.

> 4.  ":specialuse" Argument to "fileinto" Command
>
>     Usage:  fileinto [:specialuse <special-use-flag: string>]
>                      <mailbox: string>
>
> Can I suggest that all special-use-flag strings should not include the
> leading "\" character? It requires escaping in Sieve, so people might
> make mistakes when writing scripts.

I have no strong opinion about that. Disadvantage of doing that is that 
it can be confusing for people with IMAP experience and it becomes 
inconsistent with imap4flags.

Anyone else want to pitch in?

Regards,

-- 

Stephan Bosch
Senior Developer

Phone: [phone]  Fax: [phone]
Email: [e-mail]


-------------------------------------------------------------------------------------
Open-Xchange AG,  Rollnerstr. 14, 90408 Nuremberg, District Court Nuremberg HRB 24738
Managing Board: Rafael Laguna de la Vera, Carsten Dirks, Uwe Reumuth
Chairman of the Board: Richard Seibt

Dovecot Oy, Lars Sonckin Kaari 10, 02600 Espoo, Finland
Managing Director: Markku Kenttä
Chairman of the Board: Timo Sirainen
Board Member: Carsten Dirks

-------------------------------------------------------------------------------------