[GROOVY-4047] Cyclic inheritence check - scattered across multiple compilation phases

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.8-beta-1
• Fix Version/s: 1.6.8, 1.7.1, 1.8-beta-1
• Assignee: Roshan Dawrani
• Reporter: Roshan Dawrani
• Created: 11/Feb/10 03:56
• Updated: 07/Apr/10 23:46
• Resolved: 11/Feb/10 14:25

Description

Currently the check for cyclic inheritance happens at 2 places - ResolveVisitor#checkCyclicInheritence() and Verifier#visitClass().

Both are very similar checks and may be both can be done in ResolveVisitor. It's anyway something that can be caught earlier than verifier.