On 12 Jan 2006, at 08:01, John Peacock wrote:

Hanno Hecker wrote:
That was on Matt's wishlist before it can be included

That's interesting, since I was thinking last night it would make the most sense to add this whole block of code to Qpsmtpd::Plugin and you could just do

        $self->relaying_denied(DENY);

from within *any* plugin and cut to the chase. I don't see any performance benefit from making Perl dispatch a class method rather than an object method (and having in directly in the plugin object means that an inheritance walk
isn't even required).

If we start adding all this stuff to Plugin.pm it has the possibility of getting very bloated - there's so much we could just add in there for convenience. I kind of saw the DSN stuff as a helper class, something not required but useful, hence the externalisation.

Matt.

Reply via email to