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

Set the type root for some units where it wasn't yet #489

Open
wants to merge 296 commits into
base: dom-with-javac
Choose a base branch
from

Conversation

datho7561
Copy link

  • Fixes error when executing cleanups in the Eclipse IDE
  • Cleanups still don't work (they don't do anything)

@mickaelistria
Copy link

This seems to add a bunch of extra failures.

datho7561 and others added 28 commits June 14, 2024 15:50
Closes eclipse-jdt#219

Signed-off-by: David Thompson <davthomp@redhat.com>
Signed-off-by: David Thompson <davthomp@redhat.com>
Signed-off-by: David Thompson <davthomp@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>
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>
Signed-off-by: Rob Stryker <stryker@redhat.com>
Signed-off-by: David Thompson <davthomp@redhat.com>
Signed-off-by: Rob Stryker <stryker@redhat.com>
Signed-off-by: David Thompson <davthomp@redhat.com>
Setting value is "enabled" and thus has to be handled like all the other
"enabled" options.
Signed-off-by: David Thompson <davthomp@redhat.com>
fail less, log more.
Fixes eclipse-jdt#284

---------

Signed-off-by: David Thompson <davthomp@redhat.com>
Signed-off-by: Rob Stryker <stryker@redhat.com>
Co-authored-by: Rob Stryker <stryker@redhat.com>
Seems a better API than drilling down in JavacCompiler.
datho7561 and others added 20 commits June 14, 2024 15:51
Rethrow the first throwable encountered

Signed-off-by: David Thompson <davthomp@redhat.com>
Even if it's technically a field access, if all the segments are simple
names without any class names, method invocations, or keywords,
then it should be treated as a qualified name for the purpose of
converting to the JDT AST.

Signed-off-by: David Thompson <davthomp@redhat.com>
Signed-off-by: David Thompson <davthomp@redhat.com>
Signed-off-by: David Thompson <davthomp@redhat.com>
…of API to be useful

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

Cleanup ASTParser for legibility and to use the new interface

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

Add facade to CompilationUnitResolver

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

exsd doc change

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

Partial revert and cleanup

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

Clean up API changes and proposed interface

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

Undo api changes

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

Javadoc for added interface

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

More docs
as per Javadoc on interface, using start position can do the job
Signed-off-by: Rob Stryker <stryker@redhat.com>
Signed-off-by: David Thompson <davthomp@redhat.com>
eg. Adjust the source range to be correct in this case:

`for (final String asdf[][] : myListOfArrayOfArrayOfString) {}`

Signed-off-by: David Thompson <davthomp@redhat.com>
mickaelistria and others added 5 commits June 14, 2024 18:30
Signed-off-by: Rob Stryker <stryker@redhat.com>
- Small bug fix related to getKey for annotation bindings where the
  parent is sometimes not set correctly

Signed-off-by: David Thompson <davthomp@redhat.com>
- Fixes error when executing cleanups in the Eclipse IDE
- Cleanups still don't work (they don't do anything)

Signed-off-by: David Thompson <davthomp@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.

None yet

7 participants