2.5.7 regression with references?

classic Classic list List threaded Threaded
8 messages Options
Reply | Threaded
Open this post in threaded view
|

2.5.7 regression with references?

Romain Manni-Bucau
Hi guys,

using anonymous classes with 2.5.7 does not handle references properly compared to 2.5.6:

error I get:

static java.nio.file.Files.copy() is applicable for argument types: (sun.nio.fs.UnixPath, Reference)

Code implements a new SimpleFileVisitor<Path>() {...} inline and uses a reference defined before the class (~final in java).

Is it known?

Romain Manni-Bucau
@rmannibucau |  Blog | Old BlogGithub | LinkedIn | Book
Reply | Threaded
Open this post in threaded view
|

Re: 2.5.7 regression with references?

Milles, Eric (TR Tech, Content & Ops)

There were some anon. inner fixes in 2.5.7.  Can you show a small example of the AIC you are seeing an error for?




From: Romain Manni-Bucau <[hidden email]>
Sent: Friday, May 10, 2019 11:11 AM
To: [hidden email]
Subject: 2.5.7 regression with references?
 
Hi guys,

using anonymous classes with 2.5.7 does not handle references properly compared to 2.5.6:

error I get:

static java.nio.file.Files.copy() is applicable for argument types: (sun.nio.fs.UnixPath, Reference)

Code implements a new SimpleFileVisitor<Path>() {...} inline and uses a reference defined before the class (~final in java).

Is it known?

Romain Manni-Bucau
@rmannibucau |  Blog | Old BlogGithub | LinkedIn | Book
Reply | Threaded
Open this post in threaded view
|

Re: 2.5.7 regression with references?

Romain Manni-Bucau

Le ven. 10 mai 2019 à 18:35, Milles, Eric (TR Tech, Content & Ops) <[hidden email]> a écrit :

There were some anon. inner fixes in 2.5.7.  Can you show a small example of the AIC you are seeing an error for?




From: Romain Manni-Bucau <[hidden email]>
Sent: Friday, May 10, 2019 11:11 AM
To: [hidden email]
Subject: 2.5.7 regression with references?
 
Hi guys,

using anonymous classes with 2.5.7 does not handle references properly compared to 2.5.6:

error I get:

static java.nio.file.Files.copy() is applicable for argument types: (sun.nio.fs.UnixPath, Reference)

Code implements a new SimpleFileVisitor<Path>() {...} inline and uses a reference defined before the class (~final in java).

Is it known?

Romain Manni-Bucau
@rmannibucau |  Blog | Old BlogGithub | LinkedIn | Book
Reply | Threaded
Open this post in threaded view
|

Re: 2.5.7 regression with references?

Paolo Di Tommaso
Just got a weird compilation error always using Path arguments. 



p

On Fri, May 10, 2019 at 6:38 PM Romain Manni-Bucau <[hidden email]> wrote:

Le ven. 10 mai 2019 à 18:35, Milles, Eric (TR Tech, Content & Ops) <[hidden email]> a écrit :

There were some anon. inner fixes in 2.5.7.  Can you show a small example of the AIC you are seeing an error for?




From: Romain Manni-Bucau <[hidden email]>
Sent: Friday, May 10, 2019 11:11 AM
To: [hidden email]
Subject: 2.5.7 regression with references?
 
Hi guys,

using anonymous classes with 2.5.7 does not handle references properly compared to 2.5.6:

error I get:

static java.nio.file.Files.copy() is applicable for argument types: (sun.nio.fs.UnixPath, Reference)

Code implements a new SimpleFileVisitor<Path>() {...} inline and uses a reference defined before the class (~final in java).

Is it known?

Romain Manni-Bucau
@rmannibucau |  Blog | Old BlogGithub | LinkedIn | Book
Reply | Threaded
Open this post in threaded view
|

Re: 2.5.7 regression with references?

Daniel.Sun
GROOVY-9115 is fixed in 3.0.0 beta-2 and 2.5.8

Cheers,
Daniel.Sun



-----
Apache Groovy committer & PMC member
Blog: http://blog.sunlan.me 
Twitter: @daniel_sun

--
Sent from: http://groovy.329449.n5.nabble.com/Groovy-Dev-f372993.html
Apache Groovy committer & PMC member

Blog: http://blog.sunlan.me
Twitter: @daniel_sun
Reply | Threaded
Open this post in threaded view
|

Re: 2.5.7 regression with references?

Daniel.Sun
In reply to this post by Romain Manni-Bucau
I'm a bit confused because I see the build result of upgrading groovy to
2.5.7 is green:

https://travis-ci.org/Talend/component-runtime/builds/530836768?utm_source=github_status&utm_medium=notification


Would you please submit a jira ticket and provide some standalone sample
code?  Here is the link to JIRA:

https://issues.apache.org/jira/projects/GROOVY/summary


Cheers,
Daniel.Sun



-----
Apache Groovy committer & PMC member
Blog: http://blog.sunlan.me 
Twitter: @daniel_sun

--
Sent from: http://groovy.329449.n5.nabble.com/Groovy-Dev-f372993.html
Apache Groovy committer & PMC member

Blog: http://blog.sunlan.me
Twitter: @daniel_sun
Reply | Threaded
Open this post in threaded view
|

Re: 2.5.7 regression with references?

Romain Manni-Bucau


Le sam. 11 mai 2019 à 18:02, Daniel.Sun <[hidden email]> a écrit :
I'm a bit confused because I see the build result of upgrading groovy to
2.5.7 is green:

https://travis-ci.org/Talend/component-runtime/builds/530836768?utm_source=github_status&utm_medium=notification


I dropped the anonymous class in favor of a nested class to avoid the reference.




Would you please submit a jira ticket and provide some standalone sample
code?  Here is the link to JIRA:

https://issues.apache.org/jira/projects/GROOVY/summary

Will do tonight





Cheers,
Daniel.Sun



-----
Apache Groovy committer & PMC member
Blog: http://blog.sunlan.me
Twitter: @daniel_sun

--
Sent from: http://groovy.329449.n5.nabble.com/Groovy-Dev-f372993.html
Reply | Threaded
Open this post in threaded view
|

Re: 2.5.7 regression with references?

Romain Manni-Bucau

Le sam. 11 mai 2019 à 18:28, Romain Manni-Bucau <[hidden email]> a écrit :


Le sam. 11 mai 2019 à 18:02, Daniel.Sun <[hidden email]> a écrit :
I'm a bit confused because I see the build result of upgrading groovy to
2.5.7 is green:

https://travis-ci.org/Talend/component-runtime/builds/530836768?utm_source=github_status&utm_medium=notification


I dropped the anonymous class in favor of a nested class to avoid the reference.




Would you please submit a jira ticket and provide some standalone sample
code?  Here is the link to JIRA:

https://issues.apache.org/jira/projects/GROOVY/summary

Will do tonight





Cheers,
Daniel.Sun



-----
Apache Groovy committer & PMC member
Blog: http://blog.sunlan.me
Twitter: @daniel_sun

--
Sent from: http://groovy.329449.n5.nabble.com/Groovy-Dev-f372993.html