[GROOVY-4225] Wrong generation of import statements in stubs

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.7.2
• Fix Version/s: 1.7.3, 1.8-beta-1
• Component/s: Compiler
• Environment: Windows XP, Sun JDK 1.6.0_19
• Flags: Patch
• Assignee: Paul King
• Reporter: Dmytro Rud
• Created: 26/May/10 10:53
• Updated: 05/Apr/15 14:44
• Resolved: 28/May/10 11:04

Description

Single-type import declarations in Groovy files are collected/converted into type-on-demand import declarations by the stub compiler – in other words,

import my.package.Type1
import my.package.Type2

becomes

import my.package.*;

This breaks the type shadowing mechanisms described in Section 7.5.1 of the Java Language Specification (3rd edition) and leads to Java compiler errors like "cannot decide which of the classes to use".

Moreover, current implementations is based on an unordered set, thus destroying the order of import declarations.

Attached patch solves these issues by preserving both the form and the order of original import declarations.