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





------- Additional Comments From [EMAIL PROTECTED]  2004-08-11 23:15 -------
Okay, I need to get this out.  This is excellent code.  I love the patch.

I have major concerns about adding this option, though:

1. In the best case (turned off) it slows down DBM Bayes by adding 3.3% more
   time. I don't really care about the extra time and space when it is turned
   on.  In many sites, it's not even possible for users to dump the Bayes
   database so this is pure overhead.  HAMMYTOKEN and SPAMMYTOKEN also work
   without it (although I should note I think we should drop those too).

2. It's another option to support.

3. I don't think it's needed.  We had a few questions when we released pre1 and
   before that, but I've gotten used to not looking and nobody really needs to
   look.

4. It complicates the code.

If we were a commercial project, there would be no bloody way this would go
into the code.  Seriously, only nosy technical people care and apparently not
for too long judging by the 3.0 experience so far.




------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

Reply via email to