10 messages in org.apache.james.server-devRe: [jsieve] Refactoring| From | Sent On | Attachments |
|---|---|---|
| Robert Burrell Donkin | Aug 31, 2008 3:29 am | |
| Stefano Bagnara | Aug 31, 2008 6:24 am | |
| Robert Burrell Donkin | Aug 31, 2008 6:51 am | |
| Robert Burrell Donkin | Sep 1, 2008 3:03 pm | |
| Stefano Bagnara | Sep 2, 2008 3:52 am | |
| Robert Burrell Donkin | Sep 2, 2008 10:00 am | |
| Robert Burrell Donkin | Sep 4, 2008 2:50 pm | |
| Steve Brewin | Sep 5, 2008 1:24 pm | |
| Robert Burrell Donkin | Sep 6, 2008 1:45 am | |
| Stefano Bagnara | Sep 6, 2008 4:32 am |
| Subject: | Re: [jsieve] Refactoring | |
|---|---|---|
| From: | Stefano Bagnara (apa...@bago.org) | |
| Date: | Sep 2, 2008 3:52:36 am | |
| List: | org.apache.james.server-dev | |
Robert Burrell Donkin ha scritto:
On Sun, Aug 31, 2008 at 2:24 PM, Stefano Bagnara <apa...@bago.org> wrote:
<snip>
ATM it is almost impossible to make a dependency graph for jsieve because components are too much interconnected and there are too many cycles between packages and classes.
E.g: IMHO there is no clean separation between parsing and running scripts. I hope to be able to do some reorganization of the code/packages once you will have refactored it.
take another look now: it's not done but now's probably a good time to work out how to complete the work
I updated it (and cleaned junit tests), I'll look better at it later.
In the mean time I write some things that IMHO could be done there:
1) code reformat: use JAMES conventions for this ({ should not be at the beginning of lines, mainly). reformatting make it more difficult to track changes, so it is better to do that when big refactorings happen (as the first or the last change).
2) "samples" are currently in the folder but not used by mvn or ant build. They depends on james libraries (mainly the javamail-util module) and on mailet-api library. Maybe jsieve should be a multimodule too and samples should be the second module.. any other option?
3) I don't like ".junit." in the test package. I'm used to tests in the same package of the class under test.
WDYT?
Stefano