Michael Jakl wrote:
> Hi!
> 
> On Sat, May 16, 2009 at 22:52, Bernd Fondermann <bf_...@brainlounge.de> wrote:
>> For the pubsub module, I'd propose to
>>
>> + move the 'general' and 'owner' package to 'handler', or,
>> alternatively, move the handler classes they contain to two new
>> 'handler' sub-packages respectively.
> 
> +1, I wasn't happy with "general" anyways. What about .handler for the
> current "general" package and .handler.owner for the #owner namespace?

+1
I can do it, if this proves to be too cumbersome to do in a patch.

>> + rename test case classes to *TestCase or *Test so they are clearly
>> identifyable as such by their name. If you use a view where all classes
>> from one package are shown together even if they are on different dirs,
>> this would be a big help.
> 
> +1, the ant task will also profit from the "right" naming.

Great, thanks.

  Bernd

Reply via email to