Jason Tackaberry wrote:
> If config var type is a tuple, display allowed values in variable comment;

If the type is range(100) you don't want to do that. And I also have a
long list in one case, you should add a line brak after 72 chars.

> Do not comment out variables if they are using the default values -- this
> significantly improves readability in editors using syntax highlighting;

Yes it does, but it has a bad side-effect. For kaa.popcorn the
extentions for a player is a variable. If we later add a new config to
the default value the users settings won't be updated. IMHO that is
bad. I want to use defaults and adjust to the changes made to them.

> as a compromise to the previous change, include the default setting
> in the comment block;

Like above, I want to auto-update values based on the defauls.


Dischi

-- 
Take care to get what you like, otherwise you will be forced to like
what you get.

Attachment: pgphnVgSH6jyh.pgp
Description: PGP signature

-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
Freevo-cvslog mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/freevo-cvslog

Reply via email to