[GROOVY-4058] Unexpected compilation error with MapEntryExpression usage - groovy grammar issue?

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.8-beta-1
• Fix Version/s: 1.7.2, 1.8-beta-1
• Component/s: Compiler
• Assignee: Alex Tkachman
• Reporter: Roshan Dawrani
• Created: 14/Feb/10 21:19
• Updated: 07/Apr/10 23:45
• Resolved: 30/Mar/10 02:22

Description

The following code compiles fine:

import java.util.concurrent.*
class Test {
    def bar () {
        ExecutorService pool = null
        pool.submit(( Runnable ) [ run : { "some statement"; 
            //println "fishy print"
        }])
    }
}

But if I uncomment the print statement from inside the closure, like

import java.util.concurrent.*
class Test {
    def bar () {
        ExecutorService pool = null
        pool.submit(( Runnable ) [ run : { "some statement"; 
            println "fishy print"
        }])
    }
}

I get the following compilation error:

...\Try.groovy: 5: You tried to use a map entry for an index operation, this is not allowed. Maybe something should be set in parentheses or a comma is missing?
 @ line 5, column 36.
           pool.submit(( Runnable ) [ run : { "some statement"; 

Here is the AST formed in 2 cases (2nd case has the wrong AST)

Case 1: (Successful compilation)

ArgumentListExpression[
    CastExpression[
        (java.lang.Runnable) 
            MapExpression[
                MapEntryExpression(key: ConstantExpression[run], value: ClosureExpression[]{...}
                )
            ]
    ]
]

Case 2: (Failing compilation)

ArgumentListExpression[
    BinaryExpression[
        left = ClassExpression[type: java.lang.Runnable]
        operation = "["
        right = MapEntryExpression(key: ConstantExpression[run], value: ClosureExpression[]{...})]]