[jbossseam-issues] [JBoss JIRA] Updated: (JBSEAM-3460) Allow for map and set properties configured through componenets.xml to maintain there ordering

Tue Sep 30 17:11:23 EDT 2008

     [ https://jira.jboss.org/jira/browse/JBSEAM-3460?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Stuart Douglas updated JBSEAM-3460:
-----------------------------------

    Attachment: seam-collections-patch-v2.diff

I have attached another updated patch.

I think it would be great if we preserved order by default, the performance overhead would be minimal (in some situation it would even improve performance as LinkedHashMap has faster iteration). The only change that is required to the patch is to instantiate a LinkedHashMap instead of a HashMap in Component.java.

The reason why I didn't was because I did not want to break existing code that may cast injected Maps to HashMap, but I don't know how common this situation is (and anyone doing it would be relying on undocumented behavior anyway), and it would be easy to fix by manually specifying a HashMap implementation.

> Allow for map and set properties configured through componenets.xml to maintain there ordering
> ----------------------------------------------------------------------------------------------
>
>                 Key: JBSEAM-3460
>                 URL: https://jira.jboss.org/jira/browse/JBSEAM-3460
>             Project: Seam
>          Issue Type: Feature Request
>          Components: Core
>    Affects Versions: 2.1.0.GA
>            Reporter: Stuart Douglas
>         Attachments: preserve-order-patch.diff, seam-collections-patch-v2.diff, seam-collections-patch.diff
>
>
> Add a preserve-order attribute to map and set properties that would allow the ordering to be preserved.

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