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

Jason Dillon updated MNG-2930:
------------------------------

    Attachment: MNG-2930.diff

Attached patch:

 * Removes usage of JavaSource, prefer JavaClass, drops getJavaClass()
 * Promotes a few constants related to @component handling to public
 * Promotes a few methods from private to protected to allow sub-class access
 * Drops PluginDescriptor from createMojoDescriptor(), attaching to return 
value instead
 * Adds validate(MojoDescriptor) with param validation logic
 * Adds discoverClasses() to find all of the JavaClass objects to process

With these changes, extractors which can generate QDox JavaClass instances for 
their sources can use JavaMojoDescriptorExctractor as a super-class and 
override discoverClasses() and/or invoke createMojoDescriptor() if some 
additional handling needs to be done.

This works very well for my GroovyMojoDescriptorExtractor, which translates 
\*.groovy into slim Java bits to allow QDox to parse out the class, field and 
javadoc tags... and then processing of the tags is always in sync with the Java 
impl.


> Update JavaMojoDescriptorExtractor to make it more re-use friendly
> ------------------------------------------------------------------
>
>                 Key: MNG-2930
>                 URL: http://jira.codehaus.org/browse/MNG-2930
>             Project: Maven 2
>          Issue Type: Improvement
>          Components: Plugin Creation Tools
>            Reporter: Jason Dillon
>            Priority: Minor
>         Attachments: MNG-2930.diff
>
>
> Update the {{JavaMojoDescriptorExtractor}} in the {{maven-plugin-tools-java}} 
> module to make it more friendly for reusability.

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