David Nicol wrote: > For what it's worth, I like "NEXT" better than "DECLINED" too. > "DECLINED" brings to mind embarrassing failures to charge additional > purchases on maxed-out credit cards, or some other form of rejection. > "NEXT" carries no such nuance, being a familiar and functional flow > control directive.
Given that qpsmtpd is written in Perl, NEXT is perhaps a better choice than CONTINUE, but absent that I would have preferred CONTINUE. The other ones like DEFER, PASS_THROUGH, UNHANDLED etc imply things beyond just letting the next plugin run, which may not match what the plugin does. Eg: all of my plugins will set notes() for blocking reasons, leaving the actual "DENY" for a latter plugin. I consider this on the level of gratuitous changes that aren't worth doing unless it's part of a much bigger overhaul/cleanup (eg: the "Unifying QP" thread).
