Hi Ignacio!
> * The persisters extend EnhancedUserType, but they could extend
> UserType (the extra methods are not used)
>   
I am still not sure about it. The documentation of EnhancedUserType
shows, that such types can be used as identifier or discriminator, so I
assumed that this is not the case for simple UserTypes. I can't find
anything more about it in the source code and it looks like the
documentation is wrong here.
Anyway, I don't understand why we should NOT use the EnhancedUserType,
that way we can make use of the "to" and "from" string stuff hibernate
will use for some xml magic.

> * The implementation of each persister are very much the same, so they
> could extend a single abstract class and add only a couple of methods.
> The total code size shrinks to about 50%
>   
Ok, ok, I know, I am lazy. Sorry for this. I'll have a look at it.

> * I also used the Hibernate type naming pattern (DurationType,
> LocalTimeType) but that is a matter of taste and I really don't care
> much anyway.
>   
I don't know why I used the Persist* naming, I'll have a look at it too.
What do others think about this topic?

> Thanks for giving this some time, Mario
Thanks for helping making it better :-)


Ciao,
Mario


-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Joda-interest mailing list
Joda-interest@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/joda-interest

Reply via email to