[hibernate-issues] [Hibernate-JIRA] Updated: (HHH-5146) org.hibernate.util.IdentitySet.iterator() return Map.Entry instances instead of actual elements

Gail Badner (JIRA) noreply at atlassian.com
Thu Apr 22 14:14:38 EDT 2010


     [ http://opensource.atlassian.com/projects/hibernate/browse/HHH-5146?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Gail Badner updated HHH-5146:
-----------------------------

             Assignee: Gail Badner
    Affects Version/s: 3.5.1
           Issue Type: Patch  (was: Bug)

Nicklas, thanks for the patch.

I've verified that this patch would apply to 3.5.1 and that Hibernate, itself, does not use iterator() or the toArray() methods.

It would affect any applications that use IdentitySet though.

Steve, should I apply this patch to Branch_3_5 or trunk?




> org.hibernate.util.IdentitySet.iterator() return Map.Entry instances instead of actual elements
> -----------------------------------------------------------------------------------------------
>
>                 Key: HHH-5146
>                 URL: http://opensource.atlassian.com/projects/hibernate/browse/HHH-5146
>             Project: Hibernate Core
>          Issue Type: Patch
>          Components: core
>    Affects Versions: 3.5.1
>            Reporter: Nicklas Nordborg
>            Assignee: Gail Badner
>            Priority: Trivial
>         Attachments: patch.txt
>
>
> By accident I used the IdentitySet implementation in my code and got ClassCastException:s when iterating over it. It turns out that the iterator() method returns Map.entrySet().iterator() instead of Map.keySet().iterator(). There is also a similar problem with the toArray() methods. I provide a patch.

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