[
https://issues.apache.org/jira/browse/GROOVY-10140?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Paul King closed GROOVY-10140.
------------------------------
Fix Version/s: 4.0.6
Assignee: Eric Milles
Resolution: Fixed
> 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
> Assignee: Eric Milles
> Priority: Major
> Labels: varargs
> Fix For: 4.0.6
>
> 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.10#820010)
