#14 argument to -p option ignored

closed-fixed
nobody
None
5
2004-01-17
2003-10-01
Anonymous
No

Noticed this while running some test ham/spam through.
I would change the value passed to -p, but the DB didn't
change size. A DB dump revealed phrases of up to 5
words (when the man page said default was 2). That's
when I started digging around the code
:-)

BTW, from version d to e, I have gotten less false
positives - nice fix!

The relevant patch:

diff -u --recursive spamprobe-0.9e/spamprobe.cc
spamprobe-0.9e-fixed/spamprobe.cc
--- spamprobe-0.9e/spamprobe.cc Wed Sep 10
06:08:55 2003
+++ spamprobe-0.9e-fixed/spamprobe.cc Wed Oct 1
14:39:11 2003
@@ -825,7 +825,7 @@

case 'p':
if (parse_int_arg(optarg, "p", opt_value, 1, 10)) {
- factory.setMaxPhraseLength(5);
+ factory.setMaxPhraseLength(opt_value);
} else {
usage_error = true;
}

--------
Eric Schultz
eschultz@corp.untd.com

Discussion

  • Brian Burton
    Brian Burton
    2004-01-17

    • status: open --> closed-fixed