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

Paul Gier updated MPLUGIN-53:
-----------------------------

    Attachment: maven-plugin-tools-java-MPLUGIN-53-r613932.patch

It looks like the issue when away with one of the recent changes.  I'm 
attaching a test case that tests for the issue in case it recurs in the future.

> Plugin descriptor extractor crashes on certain types of Java source files
> -------------------------------------------------------------------------
>
>                 Key: MPLUGIN-53
>                 URL: http://jira.codehaus.org/browse/MPLUGIN-53
>             Project: Maven 2.x Plugin Tools
>          Issue Type: Improvement
>         Environment: Tested with Maven version 2.0.4, and 2.0.x SNAPSHOT.
>            Reporter: Paul Gier
>         Attachments: JavaMojoDescriptorExtractor.java, 
> JavaMojoDescriptorExtractor.java, 
> maven-plugin-tools-java-MPLUGIN-53-r613932.patch
>
>
> Part of my plugin includes a .java file that contains an annotation type 
> declaration.  It looks like this:
> {quote}
> package org.jboss.lang.annotation;
> import java.lang.annotation.ElementType;
> import java.lang.annotation.Retention;
> import java.lang.annotation.RetentionPolicy;
> import java.lang.annotation.Target;
> @Retention(RetentionPolicy.RUNTIME)
> @Target(ElementType.ANNOTATION_TYPE)
> public @interface Inherited {
> }
> {quote}
> When the JavaMojoDescriptorExtractor encounters this file it crashes because 
> of an ArrayIndexOutOfBoundsException.  Because the code is trying to access 
> the 1st element of a zero length array.  The attached file has a simple fix 
> where the descriptor extractor just ignores any java source file that does 
> not contain a valid class.

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

        

Reply via email to