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





------- Additional Comments From [EMAIL PROTECTED]  2004-10-06 09:49 -------
I think the massive memory hit is a big enough deal to change this API.  it's
not a widely-used plugin API, and the different semantics can be flagged easily
in a backwards-compatible way: simply rename the parameter that those tokens are
passed in as -- e.g. instead of


  $self->{main}->call_plugins("bayes_learn", { toksref => $tokens,
                                               isspam => $isspam,
                                               msgid => $msgid,
                                               msgatime => $msgatime,
                                             });

call


  $self->{main}->call_plugins("bayes_learn", { hashtoksref => $tokens,
                                               rawtoksref => $rawtoks,
                                               isspam => $isspam,
                                               msgid => $msgid,
                                               msgatime => $msgatime,
                                             });

a plugin can use "if (defined $params->{hashtoksref})" to detect this
backwards-compatibly.   (is that a word?)




------- 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