[hibernate-issues] [Hibernate-JIRA] Commented: (HHH-1646) Bad code in FastClass.equals

Scott Marlow (JIRA) noreply at atlassian.com
Wed Dec 13 11:23:05 EST 2006


    [ http://opensource.atlassian.com/projects/hibernate/browse/HHH-1646?page=comments#action_25626 ] 

Scott Marlow commented on HHH-1646:
-----------------------------------

I tried modifying org.hibernate.test.bytecode.BytecodeSuite (really org.hibernate.test.bytecode.javassist.ReflectionOptimizerTest.suite()) to recreate this bug but couldn't trigger FastClass.equals.  

I know that equals wasn't called because the call to Thread.dumpStack that I added didn't show up in the output.  Yes, I'm removing the call to dumpstack.  ;)

> Bad code in FastClass.equals
> ----------------------------
>
>          Key: HHH-1646
>          URL: http://opensource.atlassian.com/projects/hibernate/browse/HHH-1646
>      Project: Hibernate3
>         Type: Bug

>     Versions: 3.1.3, 3.2.0 cr1
>  Environment: 3.2.0
>     Reporter: Niels Harremoës
>     Assignee: Scott Marlow
>     Priority: Blocker
>      Fix For: 3.2.2
>  Attachments: FastClass.patch
>
> Original Estimate: 30 minutes
>         Remaining: 30 minutes
>
> In FastClass, the equals method is:
>   public boolean equals(Object o) {
>     if ( o == null || ( o instanceof FastClass ) ) {
>       return false;
>     }
>     return this.type.equals( ( ( FastClass ) o ).type );
> }
> There's a missing ! - the first line should be 
>  if ( o == null ||  !  ( o instanceof FastClass ) ) {

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://opensource.atlassian.com/projects/hibernate/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira





More information about the hibernate-issues mailing list