+1

Mainly for the supplied matchers and mailets, adding protected getter/setter methods.

This is a very sound and clean initiative. We should perhaps also write a 
"mailet/matcher coding best practices" wiki page.

"Also, while making this change it might be worth considering adding a prefix
to the variable names, 'field' is typical. So...
...
This has two advantages. Firstly, it avoids mistaken direct access to the
variable and secondly it makes the class JavaBean compatible..."

+1 for the first reason, but I think that it is JavaBean compatible even without the 
renaming.

Vincenzo



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

Reply via email to