GROOVY-11432: Support method references/method pointers in annotation…

[paulk-asert]

… attributes

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 68.6557%. Comparing base (6cd477b) to head (f6cdc41).
Report is 185 commits behind head on master.

Additional details and impacted files

@@                Coverage Diff                 @@
##               master      #2097        +/-   ##
==================================================
+ Coverage     68.5957%   68.6557%   +0.0600%     
- Complexity      29233      29390       +157     
==================================================
  Files            1426       1426                
  Lines          113494     113641       +147     
  Branches        19548      19597        +49     
==================================================
+ Hits            77852      78021       +169     
+ Misses          29088      29067        -21     
+ Partials         6554       6553         -1     

Files	Coverage Δ
...va/org/apache/groovy/parser/antlr4/AstBuilder.java	86.6075% <100.0000%> (-0.0235%)	⬇️
...rg/codehaus/groovy/classgen/AnnotationVisitor.java	88.6667% <100.0000%> (ø)
.../codehaus/groovy/runtime/DefaultGroovyMethods.java	73.4788% <100.0000%> (+0.0088%)	⬆️

... and 49 files with indirect coverage changes

[daniellansun]

+1

[paulk-asert]

I'll write an email to the dev list shortly and await feedback before merging. As currently designed, it works for dynamic Groovy and fails when using static compilation. This is kind of as I'd expect since we have a "dangling" method reference not associated with a functional interface. But we could also make some assumptions and use a Consumer and make it work for statically compiled code.

[paulk-asert]

Update: the current PR now uses withMethodClosure instead of the previous version which used with. This means that it works with static compilation and static type checking.