[ 
http://jira.codehaus.org/browse/MTAGLIST-46?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=196879#action_196879
 ] 

Christoph Pickl commented on MTAGLIST-46:
-----------------------------------------

@nate: okay, fixed all checkstyle errors (cpd/pmd did not find anything wrong 
with my patch). glad that there are people taking care about such things!

@anthony: the way it is implemented right now you can do this at your own. just 
add for example 
"<languageExtensions><languageExtension>groovy</languageExtension></languageExtensions>"
 to the taglist mojo configuration and you are done. same for the other file 
types -at least if they are using the same comment style... this is what we 
have to think about for the next step: support different comment styles for 
different languages...


there is still one unsolved issue i would need support with:
in the TagListReport class i've added a FIXME tag concerning reactor projects 
and the artifact handler's language (which is checked against "java"). my 
question now is now: why was this check introduced and how does this affect the 
languageExtension feature?
and also i would need someone to answer me the question concerning 
sourceFileFilter instantiation (see my comment from 10/Sep/09 03:49 PM). would 
be very nice.

~christoph

> Support for other programming languages as well
> -----------------------------------------------
>
>                 Key: MTAGLIST-46
>                 URL: http://jira.codehaus.org/browse/MTAGLIST-46
>             Project: Maven 2.x Taglist Plugin
>          Issue Type: New Feature
>         Environment: environment independent
>            Reporter: Christoph Pickl
>            Priority: Minor
>         Attachments: patch-taglist-cpickl_2009_09_02.txt, 
> patch-taglist-languageExtensions_configuration-cpickl_2009_09_10.txt
>
>
> currently only java files are supported. but as the syntax of many other 
> languages are similar to java's, it should not be that hard to adjust the 
> plugin for these other languages.
> changes which have to be made:
> 1) TagListReport#hasSources() ... change ".java"
> 2) FileAnalyser#findFilesToScan() ... change "**/*.java"
> in the attached eclipse patch you will find these changes, plus a 
> SourceFileFilter class which handles proper filtering depending on a new 
> maven configuration "languageMode" (which can now be either "java" which is 
> the default, or "flex" which looks for *.as and *.mxml files).

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