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

Implement methods in newly created type from an inherited sealed class. #1570

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rgrunber
Copy link
Contributor

.. turns out it's possible after all. Still need to clean it up a bit though.

@fbricon
Copy link
Contributor

fbricon commented Oct 15, 2020

Creates a new class as you type 😂
Oct-15-2020 18-44-05

@fbricon
Copy link
Contributor

fbricon commented Oct 15, 2020

... and the generated classes don't even implement the interface method :-)

@rgrunber
Copy link
Contributor Author

😂 thanks for verifying in IDE. I think what's happening is the main issue I had from before. The creation of the problem marker, causes the associated text edit to also be created. In our case, the text edit immediately creates the type. I'll see what I can do.

- When a non-existent type is permitted by a sealed class, the newly
  created type should implement the necessary methods
- Related eclipse-jdtls#1553

Signed-off-by: Roland Grunberg <[email protected]>
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.

2 participants