[GROOVY-9417] Make @NullCheck play nicer with @Immutable

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 4.0.0-alpha-1, 3.0.2
• Assignee: Paul King
• Reporter: Paul King
• Created: 22/Feb/20 10:18
• Updated: 09/Mar/20 03:05
• Resolved: 25/Feb/20 04:12

Description

This is a follow up to GROOVY-9406. We accidentally had generated (including constructors) having the null checks added without proper checking for things like this() and super() calls. GROOVY-9406 fixed those issues and added a includeGenerated annotation attribute but it highlighted some short-comings when using with @Immutable and @Canonical. This issue addresses those short-comings.

Issue Links

• is related to: GROOVY-9406 `VerifyError` when annotating with both @InheritConstructors and @NullCheck and calling the resulting c'tor
• links to: GitHub Pull Request #1172