Danny Angus wrote:
On 1/2/07, Stefano Bagnara <[EMAIL PROTECTED]> wrote:

In a way really similar to Danny's "Mailet API sandbox" this is thought
as an experiment to create a strawman implementation to talk about later.

Great stuff guys, this approach certainly got the debate started about
the mailet api.

If I can make a request can you keep the issues you look at separate,
so that we can discuss them one at a time? IOW try not to commit
changes which are related to two different experiments.

d.

Hi Danny, unfortunately we started the issues together because they are really tightly related.

As an example the Hook stuff is linked to the CommandHandler changes.

The CommandHandler changes have been needed to support asynchronous IO (push based). The cleanup is a core part of understanding and refactoring.

So I think we can't separate them in the current experiment. But maybe that once we'll feel more confortable with the sandbox we'll be able to better understand what we did on each "side" and maybe we'll understand that we can split back the changes or better understand what are the dependencies between the issues.

Every of that issues changed at least 1/3rd of the smtpserver package and the whole sandbox is probably almost a complete rewrite. So this will for sure require more discussion/review later.

I never intended to obfuscate what we did on that branch. The sandbox is there from only 10 days: we probably would have written something in the next weeks if not queried before. 10 days in this project are almost an "instant". I'm happy to reply to questions and to hear alternative opinions. I just wanted to not repeat past error to write long proposal before working on the issue and to understand after few weeks that the proposal was not really implementable because of concrete issues.

I consider SVN diffs and code as like as english mailing list messages for a discussion. For a developer a diff is one of the most concrete discussion to put on the table: this decrease a lot the 'I don't mean what do you mean with xxxxxxx, can you elaborate?'.

Stefano


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to