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

LSP: java.source.save.CasualDiff: Illegal values after using CodeCompletion. #3475

Merged
merged 1 commit into from
Jan 20, 2022

Conversation

dbalek
Copy link
Contributor

@dbalek dbalek commented Jan 18, 2022

After using code completion for e.g. System.out, a syntax error due to unfinished statement is usually reported. At the same time VSCode client asks for all available CodeActions (Hints) at the current line. One of the hints offers creating a static import for System.out and therefore a diff computation for such change touches the ErroneousTree representing the unfinished statement. Change to the CasualDiff was necessary to correctly handle this situation.

@dbalek dbalek added the LSP [ci] enable Language Server Protocol tests label Jan 18, 2022
@dbalek dbalek added this to the NB13 milestone Jan 18, 2022
@neilcsmith-net
Copy link
Member

Anything for consideration for NB13 needs rebasing on delivery.

@neilcsmith-net neilcsmith-net added the do not merge Don't merge this PR, it is not ready or just demonstration purposes. label Jan 18, 2022
@dbalek dbalek modified the milestones: NB13, NB14 Jan 18, 2022
@dbalek dbalek added do not merge Don't merge this PR, it is not ready or just demonstration purposes. and removed do not merge Don't merge this PR, it is not ready or just demonstration purposes. labels Jan 18, 2022
Copy link

@JaroslavTulach JaroslavTulach left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let' do it!

@dbalek dbalek merged commit f1a3538 into apache:master Jan 20, 2022
@dbalek dbalek deleted the dbalek/GR-36240 branch January 20, 2022 08:09
lkishalmi pushed a commit to lkishalmi/netbeans that referenced this pull request Jan 20, 2022
jhorvath pushed a commit to jhorvath/netbeans that referenced this pull request Feb 2, 2022
jhorvath pushed a commit to jhorvath/netbeans that referenced this pull request Feb 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
LSP [ci] enable Language Server Protocol tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants