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

[Max R. A. (JIRA) <no...@at...>]

    [ http://opensource.atlassian.com/projects/hibernate/browse/HHH-1646?pa=
ge=3Dcomments#action_23695 ]=20

Max Rydahl Andersen commented on HHH-1646:
------------------------------------------

greg: no, null instanceof X is always true.

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

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

--=20
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