You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@spamassassin.apache.org by bu...@bugzilla.spamassassin.org on 2004/01/30 03:31:37 UTC

[Bug 2987] Put user config options at top of Conf.pm

http://bugzilla.spamassassin.org/show_bug.cgi?id=2987





------- Additional Comments From quinlan@pathname.com  2004-01-29 17:36 -------
Subject: Re:  New: Put user config options at top of Conf.pm

> Basically, most users will only ever change a small number of options.
> So instead of having them spread throughout the M::SA::Conf POD docs,
> let's put the most common one up at the top of the Conf.pm file.  That
> way if they're looking for "score", or a bayes tweak or something, it
> won't require going through a bunch of options they either can't use
> or otherwise don't care about (or otherwise shouldn't be touching! ;)

I think we should perhaps separate the documentation somewhat from the
implementation.  Perhaps make Conf.pm just the parser and reader and put
basic options into "Basic", and advanced options into "Advanced".

Also, moving large blocks of stuff around in Conf.pm seems a bit
wasteful, I'd still like us to do something like Malte's reorganization
first (proposed some time ago).  The parsing could be made significantly
faster by hashing on the first word in the line instead of doing a
series of regexp if-then-elses.

Daniel





------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.