artur-shaik / vim-javacomplete2

DEPRECATED in favor of jc.nvim
Vim License
974 stars 83 forks source link

Importing missing imports doesn't take inner classes into account #366

Closed ym1234 closed 5 years ago

ym1234 commented 6 years ago

For example, Say I have Outer class called Computer and it has a inner class called Memory, JCimportsAddMissing will try to find a class called Memory and import it.

Actual behavior (Required!)

Doesn't take inner classes into account

Expected behavior (Required!)

Should take inner classes into account

The steps to reproduce actual behavior (Required!)

  1. Create a class
  2. Create an inner class
  3. try to import missing imports

Environment (Required!)

Q&A

artur-shaik commented 6 years ago

As I remember it's old issue related to inability of javaparser to fetch inner classes from source. We should try to check new versions of javaparser. I bet they fixed it.

ym1234 commented 6 years ago

How could I test if the newer version of javaparser fixes problem?

artur-shaik commented 6 years ago

If you use maven you can change version of javaparser in pom file of javavi project. But anyway, I think it will require javacomplete2 code modification as well.

On 01-31 (17:53), ym123 wrote:

How could I test if the newer version of javaparser fix problem?

— You are receiving this because you commented. Reply to this email directly, [1]view it on GitHub, or [2]mute the thread.

Reverse link: [3]unknown

References

Visible links

  1. https://github.com/artur-shaik/vim-javacomplete2/issues/366#issuecomment-362015244
  2. https://github.com/notifications/unsubscribe-auth/AEozu3O2xA4_5eny-6uZ-TnBpNZ0BYcrks5tQKiJgaJpZM4RzYLl
  3. https://github.com/artur-shaik/vim-javacomplete2/issues/366#issuecomment-362015244

-- Best regards, Artur Shaikhullin