Stefano Bagnara wrote:
> Noel J. Bergman wrote:
> > The intent -- at least from some of us -- for the handler chain
> > was not to keep adding features into core handlers, but to add
> > additional handlers. So far, I keep seeing more optional
> > features added to core handlers.
> >
> > Comments?
> I applied the following patches:
> I applied them in the basic commands because:
> I expect that other committers review the commits descriptions and
> the related JIRA issues and I can revert any commit if needed. I
> though this "optimistic" workflow is faster than voting for each
> single commit.
Agreed on all of the above. As you might notice, I have been reviewing the
commits, was not asking for them to be reverted, and was raising a general
issue based upon a recent pattern.
Both you and Norman raise the issue that "creating multiple optional
handlers would have been code duplication and more difficult to mantain in
those cases." That suggests looking to see if we can factor out more common
code, such that the handlers are more function specific and less common
scafolding.
I certainly don't feel that the current form needs to be, or indeed is going
to be, stable. What we have is a first cut at a major functional area.
> We should decide what to do with additional mailets/matcher/handlers in
> general: we currently don't have a repository and a website where to
> publish those additional stuff but we can't continue adding each new
> mailet to the james core.
Good timing. I was going to suggest that it might be good to provide a
second package of matchers and mailets, not part of our core, to show people
how to do those. Also, it would provide us a place to put some
non-essential samples, or even deprecated matchers/mailets, such as the old
list server. We could extend and generalize that idea, perhaps. We could
also revisit the idea of handler packages to see if we can avoid having to
specify each one separately.
--- Noel
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
