http://bugzilla.spamassassin.org/show_bug.cgi?id=4176
------- Additional Comments From [EMAIL PROTECTED] 2005-05-05 17:41 -------
> I don't like the idea of having two different behaviours depending on
> how the message object is created...
That's what's bothering me about this. The issue for me is what is parse()
defined as doing? Is it a class method or an object method?
I just had an idea... The problem is that configuration options are associated
with the Mail::SpamAssassin instance. We had to do that in order to have
per-user options. But we have changed things since then to have a system
configuration that is loaded once and then extending $main->{conf} with per-user
options.
So why can't the base {conf} options be in a class variable instead of an
instance variable? It would be accessed via a class method, which means we would
not have to pass through the main object to all those constructors.
Is there a reason why Mail::SpamAssassin::Conf-new is called when a new instance
of Mail::SpamAssassin is created and not when the class is loaded?
Would that simplify how we handle the comfiguration options?
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
