[jira] Created: (QDOX-163) unordered exception list

unordered exception list
------------------------

                 Key: QDOX-163
                 URL: http://jira.codehaus.org/browse/QDOX-163
             Project: QDox
          Issue Type: Bug
    Affects Versions: 1.9.1
         Environment: jdk5
            Reporter: yusheng
             Fix For: 1.10


Assume there is a class

public class A {
      public void doSomething() throws Exception1, Exception2, ExceptionN {
           // code
      }
}

Suppose Exception1, and Exception2 are all subclasses of ExceptionN. The exception list returned is an unordered set.  I am parsing this class to generate another class, which needs to handle the exceptions of method doSomething(). Sometimes the generated class has compile error. The generated code may looks like

catch (ExceptionN e) {

} catch (Exception1 e) {
}

Although the method doSomething exception list is not necessary as only more general exception is enough (doSomething() throws  ExceptionN.

This kind class is perfectly legal and exists in one production class.

I would suggest change the set to list in class MethodDef (List exceptions = new ArrayList();)




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

     [ http://jira.codehaus.org/browse/QDOX-163?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Robert Scholte resolved QDOX-163.
---------------------------------

      Assignee: Robert Scholte
    Resolution: Fixed

Exceptions was a Set, so I think it's better just to change it's implementation. Now it's a LinkedHashSet, which should do the trick as well.

> unordered exception list
> ------------------------
>
>                 Key: QDOX-163
>                 URL: http://jira.codehaus.org/browse/QDOX-163
>             Project: QDox
>          Issue Type: Bug
>    Affects Versions: 1.9.1
>         Environment: jdk5
>            Reporter: yusheng
>            Assignee: Robert Scholte
>             Fix For: 1.10
>
>
> Assume there is a class
> public class A {
>       public void doSomething() throws Exception1, Exception2, ExceptionN {
>            // code
>       }
> }
> Suppose Exception1, and Exception2 are all subclasses of ExceptionN. The exception list returned is an unordered set.  I am parsing this class to generate another class, which needs to handle the exceptions of method doSomething(). Sometimes the generated class has compile error. The generated code may looks like
> catch (ExceptionN e) {
> } catch (Exception1 e) {
> }
> Although the method doSomething exception list is not necessary as only more general exception is enough (doSomething() throws  ExceptionN.
> This kind class is perfectly legal and exists in one production class.
> I would suggest change the set to list in class MethodDef (List exceptions = new ArrayList();)

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

     [ http://jira.codehaus.org/browse/QDOX-163?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Paul Hammant updated QDOX-163:
------------------------------

    Fix Version/s:     (was: 1.10)
                   1.9.2

> unordered exception list
> ------------------------
>
>                 Key: QDOX-163
>                 URL: http://jira.codehaus.org/browse/QDOX-163
>             Project: QDox
>          Issue Type: Bug
>    Affects Versions: 1.9.1
>         Environment: jdk5
>            Reporter: yusheng
>            Assignee: Robert Scholte
>             Fix For: 1.9.2
>
>
> Assume there is a class
> public class A {
>       public void doSomething() throws Exception1, Exception2, ExceptionN {
>            // code
>       }
> }
> Suppose Exception1, and Exception2 are all subclasses of ExceptionN. The exception list returned is an unordered set.  I am parsing this class to generate another class, which needs to handle the exceptions of method doSomething(). Sometimes the generated class has compile error. The generated code may looks like
> catch (ExceptionN e) {
> } catch (Exception1 e) {
> }
> Although the method doSomething exception list is not necessary as only more general exception is enough (doSomething() throws  ExceptionN.
> This kind class is perfectly legal and exists in one production class.
> I would suggest change the set to list in class MethodDef (List exceptions = new ArrayList();)

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