[ 
http://jira.codehaus.org/browse/MASPECTJ-90?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=252343#action_252343
 ] 

Robert Scholte commented on MASPECTJ-90:
----------------------------------------

The key issue is that you want to keep the order of the dependencies.
A {{TreeSet}} will give you a *sorted* List with unique elements instead of an 
ordered List.
{quote}[Class 
TreeSet|http://download.oracle.com/javase/6/docs/api/java/util/TreeSet.html]
A NavigableSet implementation based on a TreeMap. The elements are ordered 
using their natural ordering, or by a Comparator provided at set creation time, 
depending on which constructor is used. 
{quote}

A {{HashSet}} will give no guarantees at all:
{quote}[Class 
HashSet|http://download.oracle.com/javase/6/docs/api/java/util/HashSet.html]
This class implements the Set interface, backed by a hash table (actually a 
HashMap instance). It makes no guarantees as to the iteration order of the set; 
in particular, it does not guarantee that the order will remain constant over 
time. This class permits the null element. 
{quote}

Since you want to keep the order of the dependencies as listed in the pom (this 
will also be the order in which they'll be available on the classpath) you need 
to use an ordered list, meaning an ArrayList or a LinkedList. 


> Classhpath on aspectJ plugin doesn't respect the dependencies order
> -------------------------------------------------------------------
>
>                 Key: MASPECTJ-90
>                 URL: http://jira.codehaus.org/browse/MASPECTJ-90
>             Project: Mojo AspectJ Plugin
>          Issue Type: Bug
>    Affects Versions: 1.2, 1.3, 1.4
>         Environment: *
>            Reporter: Marvin Froeder
>            Assignee: Robert Scholte
>             Fix For: 1.4
>
>
> AspectJ classpath is completely out of order.
> The usage of TreeSet and HashSet on AjcHelper is causing that.
> Patch for it attached

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

        

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email


Reply via email to