eric-milles opened a new pull request #1330:
URL: https://github.com/apache/groovy/pull/1330


    - `Thread.currentThread().getContextClassLoader()` was used starting with 
46b1fface0b17cf41f9681894bcb8d78fc831258 -- commit does not state that this 
parentage is required for any specific reason; fallback loader (the one that 
loaded `ProcessingUnit`) seems like a better option for Groovy.
   
    - `StaticTypeCheckingSupport.evaluateExpression(...)` relies on loader 
created by this block; it does not transfer loader(s) from it's context.
   
   
   This came about for me because IDE is performing content assist 
asynchronously now and processing `ClosureParams`/`DelegatesTo` metadata relies 
on `evaluateExpression`.


----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to