You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@spamassassin.apache.org by Justin Mason <jm...@jmason.org> on 2006/11/16 10:40:34 UTC

Re: svn commit: r475485 - in /spamassassin/branches/jm_re2c_hacks/lib/Mail/SpamAssassin/Plugin: Check.pm RabinKarpBody.pm Rule2XSBody.pm

Michael Parker writes:
> Ugh, hardcoded package names.
> 
> You'll notice that I did away with hardcoded package names in the
> Check.pm rules functions.  This means that it can be easily subclassed
> and just work.  Not sure what the best solution for this case is but
> lets try to avoid hardcoded stuffs.

yeah, agreed -- this is still pretty hacky in this branch and that
will need to be fixed.

Ideally those classes won't call into Mail::SpamAssassin::Plugin::Check;
instead they'd have their *own* method which generates the one_line_body
match methods *on their own class* I think.

--j.