groovy-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Eric Milles (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (GROOVY-5961) Variable scope not checked properly for AIC in static method
Date Sun, 14 Apr 2019 14:41:00 GMT

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

Eric Milles commented on GROOVY-5961:
-------------------------------------

Something more is required.  A case that should fail no longer produces a compiler error:
{code:groovy}
class C {
  int count
  static def m() {
    new LinkedList() {
      def get(int i) {
        count += 1 // expect: Apparent variable 'count' was found in a static scope but doesn't
refer to a local variable, static field or class.
        super.get(i)
      }
    }
  }
}
{code}

> Variable scope not checked properly for AIC in static method
> ------------------------------------------------------------
>
>                 Key: GROOVY-5961
>                 URL: https://issues.apache.org/jira/browse/GROOVY-5961
>             Project: Groovy
>          Issue Type: Sub-task
>          Components: Compiler
>    Affects Versions: 2.1.0
>            Reporter: C├ędric Champeau
>            Priority: Major
>
> The following code will fail:
> {code}
> static void foo() {
>     new LinkedList() {
>         int index
>         Object get(int i) { super.get(index++) }
>     }
> }
> {code}
> The error is:
> {noformat}
> Apparent variable 'index' was found in a static scope but doesn't refer to a local variable,
static field or class. Possible causes:
> You attempted to reference a variable in the binding or an instance variable from a static
context.
> You misspelled a classname or statically imported field. Please check the spelling.
> You attempted to use a method 'index' but left out brackets in a place not allowed by
the grammar.
>  at line: 5, column: 39
> {noformat}
> But obviously, {{index}} is not accessed from a static context here.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Mime
View raw message