[GROOVY-7958] Incorrect parsing of comma-separated variable declaration as single statement after if/while/for

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.4.8
• Assignee: Paul King
• Reporter: Paul King
• Created: 05/Oct/16 07:31
• Updated: 01/Feb/17 23:18
• Resolved: 05/Oct/16 23:04

Description

Currently during parsing a single (comma-separated) variable declaration is turned into multiple statements on the fly. If this occurs when such a statement is the single statement after an if/while/for, then only the first statement is picked up and the next is left dangling within the AST tree. For 'if' it becomes the else statement! while in the 'for/while' cases I think it is ignored. So, the following code doesn't work as expected:

int xNext = 0, yNext = 0
if (false) int x = xNext++, y = yNext++
assert xNext == 0 && yNext == 0
if (true) int x = xNext++, y = yNext++
assert xNext == 1 && yNext == 1

This is very much an edge case (what good is a declaration if you then have no block in which to use it?) but is still worth fixing.

Issue Links

• links to: GitHub Pull Request #437