paulk-asert commented on pull request #1318: URL: https://github.com/apache/groovy/pull/1318#issuecomment-660549463
This seems okay. It doesn't seem to break the existing import of static classes using "import static", which while possibly unexpected from a Java point-of-view, we have supported for a long time. I'll note that your patch doesn't extend to another "exotic" case we currently support. ``` import java.util.Map as SomeMap import static SomeMap.Entry as SomeEntry assert SomeEntry instanceof Class ``` works but not without the static. It is also somewhat unusual but accounts for aliases to some degree and we have supported it for along time too. We can cover that off separately. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
