On Mon, 25 Aug 2008 13:48:13 +0200, Sanne Grinovero <[EMAIL PROTECTED]> wrote:

A) the latest method "isPresentInListeners";
especially because the current code isn't checking
all listeners the same way, I suppose because you merged the now deprecated
FullTextIndexCollectionEventListener with the FullTextIndexEventListener.
I'd use isAssignableFrom, but there's a comment there explicitly telling me not to do.
Regarding the inconsistent check I would say that this is just a mistake. The checks should be consistent and I don't see a reason why PostCollectionUpdateEventListener should be treated differently to the others. I also think 'isAssignableFrom' is the way to go. Emmanunel is online tomorrow. Maybe he can enlighten us regarding this comment.

B) I'm writing a trivial test, is there any special configuration to test for?
Of course the latest configuration - with the latest annotations depedency - has to work. Other tests might be harder since they depend on different versions of Annotations and Search. You can test whether an older version of Annotations still works with the latest of Search and vice versa.

--Hardy


_______________________________________________
hibernate-dev mailing list
hibernate-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/hibernate-dev

Reply via email to