Skip to content
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

For Unnamed Classes, generate the .class with the same name as the .java #102

Merged

Conversation

datho7561
Copy link

@datho7561 datho7561 commented Feb 29, 2024

Using the <unnamed_class$MyUnnamedClass> name is bad, because then the generated .class file is invalid and can't be run.

Current blocking problems:

@datho7561 datho7561 force-pushed the generate-classfile-with-same-name branch from 8f2aca6 to 38f99be Compare March 6, 2024 14:01
@datho7561 datho7561 force-pushed the generate-classfile-with-same-name branch from 67bcb4e to 466ad3e Compare March 13, 2024 17:03
@datho7561 datho7561 marked this pull request as ready for review March 13, 2024 17:04
@datho7561
Copy link
Author

datho7561 commented Mar 13, 2024

I rewrote one of the tests because of eclipse-jdt#2156 (Completion fails when hiding enclosing type)

@datho7561
Copy link
Author

@rgrunber @fbricon this is ready for review

@datho7561 datho7561 force-pushed the generate-classfile-with-same-name branch 2 times, most recently from 30626c5 to 37130ac Compare March 18, 2024 16:02
…`.java`

- Provide and API to check if an IType is an implicitly declared class

Signed-off-by: David Thompson <davthomp@redhat.com>
@datho7561 datho7561 force-pushed the generate-classfile-with-same-name branch from 37130ac to bb49572 Compare March 18, 2024 16:13
@rgrunber rgrunber merged commit f24d2b7 into eclipse-jdtls:develop Mar 18, 2024
3 checks passed
@datho7561 datho7561 deleted the generate-classfile-with-same-name branch March 18, 2024 17:54
rgrunber added a commit to rgrunber/eclipse.jdt.ls that referenced this pull request Mar 18, 2024
- See eclipse-jdtls/eclipse-jdt-core-incubator#102

Signed-off-by: Roland Grunberg <rgrunber@redhat.com>
rgrunber added a commit to eclipse-jdtls/eclipse.jdt.ls that referenced this pull request Mar 18, 2024
- See eclipse-jdtls/eclipse-jdt-core-incubator#102

Signed-off-by: Roland Grunberg <rgrunber@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants