Skip to content

Fix two errors related to Code select and local variables#1457

Open
robstryker wants to merge 1344 commits into
eclipse-jdtls:dom-with-javacfrom
robstryker:codeSelectRelatedToLocalVar
Open

Fix two errors related to Code select and local variables#1457
robstryker wants to merge 1344 commits into
eclipse-jdtls:dom-with-javacfrom
robstryker:codeSelectRelatedToLocalVar

Conversation

@robstryker
Copy link
Copy Markdown

No description provided.

@robstryker robstryker force-pushed the codeSelectRelatedToLocalVar branch 2 times, most recently from 3d3f9c2 to ae783f0 Compare May 22, 2025 21:09
@robstryker
Copy link
Copy Markdown
Author

This PR needs rework because it depends on the a Javac Class using the same implementation to create the Local Variable that jdt.core in another spot, and they don't perfectly align.

This PR attempts to unify the logic but I forgot we no longer build our own jdt.core. I'll finish this one tomorrow.

@robstryker robstryker force-pushed the codeSelectRelatedToLocalVar branch 2 times, most recently from 4f4fa88 to 94e15e0 Compare May 27, 2025 17:55
@robstryker
Copy link
Copy Markdown
Author

This PR won't pass until eclipse-jdt#4054 is merged upstream.

Rob Stryker and others added 19 commits July 31, 2025 12:28
…ents02

Signed-off-by: Rob Stryker <stryker@redhat.com>
…ts06

Signed-off-by: Rob Stryker <stryker@redhat.com>
… other

Signed-off-by: Rob Stryker <stryker@redhat.com>
Signed-off-by: Rob Stryker <stryker@redhat.com>
Signed-off-by: Rob Stryker <stryker@redhat.com>
…guments05 and one other

Signed-off-by: Rob Stryker <stryker@redhat.com>
Signed-off-by: Rob Stryker <stryker@redhat.com>
Signed-off-by: Rob Stryker <stryker@redhat.com>
Signed-off-by: Rob Stryker <stryker@redhat.com>
Signed-off-by: Rob Stryker <stryker@redhat.com>
…patterns

Signed-off-by: Rob Stryker <stryker@redhat.com>
…ly wrong

  Fixes 15

Signed-off-by: Rob Stryker <stryker@redhat.com>
…are parseable

Signed-off-by: Rob Stryker <stryker@redhat.com>
Signed-off-by: Rob Stryker <stryker@redhat.com>
Signed-off-by: Rob Stryker <stryker@redhat.com>
Signed-off-by: Rob Stryker <stryker@redhat.com>
Signed-off-by: Rob Stryker <stryker@redhat.com>
…lent tests

Signed-off-by: Rob Stryker <stryker@redhat.com>
@robstryker robstryker force-pushed the codeSelectRelatedToLocalVar branch from 94e15e0 to e1b6bb5 Compare August 11, 2025 18:10
@robstryker robstryker force-pushed the dom-with-javac branch 3 times, most recently from b1b4e0e to 309f390 Compare September 8, 2025 20:48
@robstryker robstryker force-pushed the dom-with-javac branch 2 times, most recently from 5738537 to 9962d57 Compare December 2, 2025 20:28
@robstryker robstryker force-pushed the dom-with-javac branch 2 times, most recently from dd7aa18 to fbe5e1a Compare March 6, 2026 18:59
@robstryker robstryker force-pushed the dom-with-javac branch 2 times, most recently from 2fcc4f8 to b7970ce Compare April 23, 2026 17:03
@robstryker robstryker force-pushed the dom-with-javac branch 2 times, most recently from 86c88a2 to 4ccbd95 Compare May 7, 2026 17:06
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.

5 participants