[jira] [Commented] (IO-531) make it more esay-to-use: org.apache.commons.io.FileUtils.listFiles(File, IOFileFilter, IOFileFilter)

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[jira] [Commented] (IO-531) make it more esay-to-use: org.apache.commons.io.FileUtils.listFiles(File, IOFileFilter, IOFileFilter)

JIRA jira@apache.org

    [ https://issues.apache.org/jira/browse/IO-531?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15977384#comment-15977384 ]

Bernd Eckenfels commented on IO-531:
------------------------------------

The test here implements the (null, trueFilter) case, but it uses FileFilterUtils.trueFileFilter, should the javadoc use this instead?

https://github.com/apache/commons-io/blob/master/src/test/java/org/apache/commons/io/FileUtilsListFilesTestCase.java#L161

> make it more esay-to-use: org.apache.commons.io.FileUtils.listFiles(File, IOFileFilter, IOFileFilter)
> -----------------------------------------------------------------------------------------------------
>
>                 Key: IO-531
>                 URL: https://issues.apache.org/jira/browse/IO-531
>             Project: Commons IO
>          Issue Type: Improvement
>          Components: Utilities
>    Affects Versions: 2.5
>            Reporter: Hao Liu
>         Attachments: Commons IOIO-531.pdf
>
>
> when I only want to filter the directories, it should be better to allow me to set the second parameter to null or I have to implement the org.apache.commons.io.filefilter.IOFileFilter interface with nothing functionally task to do.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
Loading...