s4-param: +foo syntax

[Jelmer Vernooij]

On Wed, 2010-12-08 at 11:14 +0100, Stefan (metze) Metzmacher wrote:
> >  > Wouldn't it be clearer to use 'server services += openchange'?
> > 
> > we discussed that, but were concerned about moving away from standard
> > .ini file syntax in case other parsers/editors are used.
This change already moves us away from the standard syntax. Existing
parsers/editors will get the value wrong (silently), rather than
failing. 

> >  > '+' might be a valid character in the list items.
> > 
> > note that we created two types of lists, P_LIST and P_CMDLIST. P_LIST
> > is only used for things where + and - makes sense.
This is confusing, as users will try to use it on some of the other
parameters as well and find that it does not work.

+= is also commonly used in other places to mean "extend with", so it
would be less confusing.

Cheers,

Jelmer
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20101209/84f95019/attachment.pgp>