I opened GERONIMO-4639 and fixed this in trunk and branches 2.1

I couldn't understand the nested iff statements too well so I just replaced it with what idea generates for equals :-)

This is a component, not part of geronimo itself. We'll need to push a release of at least the 2.1 branch.

thanks
david jencks


On May 20, 2009, at 5:58 AM, Bert_nor wrote:


java.lang.NullPointerException
       at
org.apache.geronimo.connector.outbound.MultiPoolConnectionInterceptor $SubjectCRIKey.equals(MultiPoolConnectionInterceptor.java:193)

source:
return hashcode == o.hashcode &&
                   (subject == null ? o.subject == null :
subject.equals(o.subject) &&
                   cri == null ? o.cri == null : cri.equals(o.cri));

need brackets:
return hashcode == o.hashcode &&
                   (subject == null ? o.subject == null :
subject.equals(o.subject) &&
                   ( cri == null ? o.cri == null : cri.equals(o.cri))
                   );

--
View this message in context: 
http://www.nabble.com/NullPointerException-MultiPoolConnectionInterceptor%24SubjectCRIKey.equals-tp23634940s134p23634940.html
Sent from the Apache Geronimo - Users mailing list archive at Nabble.com.

Reply via email to