-
Notifications
You must be signed in to change notification settings - Fork 194
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Missing code assist in static method with @CompileStatic #371
Comments
ready to test |
Hi Eric, I tested this with 2.9.2.xx-201711160311-e46, but it works partially. This discrepancy on code assist "right after the dot" working, but not "after some letters have been typed", is not rare in Greclipse and is really annoying indeed :-( It makes the whole thing feel "fragile" to use. |
From a user's perspective, this may all seem like the same thing. But under the covers, recognizing "url.|" (i.e. no expression after the dot) and "url.a|" are actually two different things. The ASTs are quite different. It does not always follow that one will work if the other does. This may seem slow and painful, but each issue needs to be rooted out on its own. Adding @CompileStatic in 2.0 created a lot of tech debt all at once.
|
I understand. Do you wish me to open a separate issue for this or let we keep this bug report as the "user perspective" on proper autocompletion in this scenario? |
I am working on a fix. I thought I had tried this out before checking in last time, but I must have been trying that case with @CompileStatic off. Please understand the level of effort required to get it to this point. When I picked up development of Groovy-Eclipse, @CompileStatic was barely handled by the inferencing engine.
|
Don't worry Eric, no negligence by your side, you're doing a great job, really! |
ready to test |
Tested again with 2.9.2.xx-201711162156-e46, now all seems to work fine, even camel-case completion! 👍 |
Follow-up to #360.
Consider the following Java class:
And the following Groovy class:
Invoke code assist at "|": no suggestion at all is shown.
Please note that if you remove
@CompileStatic
or if you replace it with@TypeChecked
, it works fine.The text was updated successfully, but these errors were encountered: