[jira] [Resolved] (GROOVY-9240) Better signature for ResourceGroovyMethods.traverse(File, Map, Closure) (and overloadings)

["Daniel Sun (Jira)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/GROOVY-9240?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Daniel Sun resolved GROOVY-9240.
--------------------------------
    Fix Version/s: 3.0.0-beta-4
         Assignee: Eric Milles
       Resolution: Fixed

The proposed PR was merged. Thanks!

> Better signature for ResourceGroovyMethods.traverse(File, Map<String, Object>, Closure) (and overloadings)
> ----------------------------------------------------------------------------------------------------------
>
>                 Key: GROOVY-9240
>                 URL: https://issues.apache.org/jira/browse/GROOVY-9240
>             Project: Groovy
>          Issue Type: Improvement
>          Components: groovy-jdk
>    Affects Versions: 2.5.8
>            Reporter: Mauro Molinari
>            Assignee: Eric Milles
>            Priority: Major
>             Fix For: 3.0.0-beta-4
>
>          Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> When you try to perform the following invocation in a statically checked/compile Groovy class you'll get a compilation error:
> {code:groovy}
> myFolder.traverse([
> 			type: FileType.FILES,
> 			nameFilter: ~/.*\.(?i)pdf/
> 		]) { println it }
> {code}
> The error is: 
> {noformat}
> Groovy:[Static type checking] - Cannot call java.io.File#traverse(java.util.Map <java.lang.String, java.lang.Object>, groovy.lang.Closure) with arguments [java.util.LinkedHashMap <java.lang.String, java.io.Serializable>, groovy.lang.Closure] 
> {noformat}
> I think that more flexible and static type checker-friendly signatures for the {{traverse}} methods would be:
> {{ResourceGroovyMethods.traverse(File, Map<String, ?>, Closure)}}
> {{ResourceGroovyMethods.traverse(File, Map<String, ?>)}}



--
This message was sent by Atlassian Jira
(v8.3.2#803003)