[jira] [Commented] (GROOVY-10140) No compiler error for invalid transient method modifier

["Eric Milles (Jira)" <ji...@apache.org>]

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

Eric Milles commented on GROOVY-10140:
--------------------------------------

https://github.com/apache/groovy/commit/ef4044db62ddb9df7e89d8c4e03370319cfb7156

> No compiler error for invalid transient method modifier
> -------------------------------------------------------
>
>                 Key: GROOVY-10140
>                 URL: https://issues.apache.org/jira/browse/GROOVY-10140
>             Project: Groovy
>          Issue Type: Bug
>    Affects Versions: 2.5.14, 3.0.8, 4.0.0-alpha-3
>            Reporter: Eric Milles
>            Priority: Major
>              Labels: varargs
>          Time Spent: 2.5h
>  Remaining Estimate: 0h
>
> Consider the following:
> {code:groovy}
> class C {
>   transient void m() {
>     println 'should not compile'
>   }
> }
> new C().m()
> {code}
> Groovy happily compiles and executes this code.  It should emit and error for the invalid modifier "transient" on the method declaration.  Similar code in Java produces the following error:
> {code}
> Illegal modifier for the method m; only public, protected, private, abstract, static, final, synchronized, native & strictfp are permitted
> {code}
> transient overlaps the modifier bit for varargs so this can cause difficulties in joint compilation scenarios.  -Also the modifier printing code leveraged from {{MethodNode#getText}} does not consider the method context and prints "transient" for "def foo(... args) {}".-



--
This message was sent by Atlassian Jira
(v8.20.1#820001)