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

Nate Stoddard commented on MTAGLIST-46:
---------------------------------------

Great feature request Christoph, and I like the filename check logic 
encapsulation within your SourceFileFilter.

I do have a suggestion.  If Taglist uses this "mode" approach, we will need to 
update the source code and perform a release for every new language or new file 
extension that is wanted.  Also some developers use non-standard filename 
extensions, so what if we move the control to the user?  e.g.

  private List languageExtensions;

  <languageExtensions>
    <languageExtension>as</languageExtension>
    <languageExtension>mxml</languageExtension>
  </languageExtensions>

This list could be passed and used by your SourceFileFilter.



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