You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@groovy.apache.org by "Eric Milles (Jira)" <ji...@apache.org> on 2022/10/01 17:26:00 UTC
[jira] [Resolved] (GROOVY-10770) Multiple @CompileStatic annotations don't combine extensions
[ https://issues.apache.org/jira/browse/GROOVY-10770?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Eric Milles resolved GROOVY-10770.
----------------------------------
Fix Version/s: 5.0.0-alpha-1
Resolution: Fixed
https://github.com/apache/groovy/commit/168ac8f269620f3d2bef85dbabd364823f8f2dec
> Multiple @CompileStatic annotations don't combine extensions
> ------------------------------------------------------------
>
> Key: GROOVY-10770
> URL: https://issues.apache.org/jira/browse/GROOVY-10770
> Project: Groovy
> Issue Type: Bug
> Components: Static Type Checker
> Affects Versions: 4.0.5
> Reporter: Christopher Smith
> Assignee: Eric Milles
> Priority: Major
> Fix For: 5.0.0-alpha-1
>
>
> When multiple levels of {{@CompileStatic}} are present, various type-checking extensions can be specified independently, but the total list isn't aggregated:
> {code:groovy}
> @CompileStatic(extensions = 'some-extension.groovy')
> class MyClass {
> @CompileStatic(extensions = 'erroneously-skipped.groovy')
> def myMethod() {}
> }
> {code}
> In such a case, all extensions should be applied collectively, inmost first. It might also make sense at some point to add some configuration to undo/disable other extensions, but for now composition is the piece that's not working correctly.
> I speculate that unified handling for {{@CompileStatic}} and {{@TypeChecked}} might be needed to make the various flavors all work together as expected by end users.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)