Skip to content

Conversation

datho7561
Copy link
Contributor

Update the flags to match https://github.com/eclipse-jdtls/eclipse-jdt-core-incubator/blob/dom-with-javac/org.eclipse.jdt.core.javac/META-INF/p2.inf (I've reordered some flags to match that file).

This opens a bunch of jdk internals that the jdt-javac work uses, and also sets the flags to enable the javac-based search.

@datho7561
Copy link
Contributor Author

I need to double check that I've got all the correct flags still before this is ready for review. Hopefully I'll get that done first thing tomorrow

@datho7561 datho7561 marked this pull request as draft October 6, 2025 20:40
@datho7561 datho7561 force-pushed the update-flags-for-jdt-javac branch from ac31b4f to 9bc69a2 Compare October 8, 2025 18:29
@datho7561 datho7561 marked this pull request as ready for review October 8, 2025 18:29
Update the flags to match
https://github.com/eclipse-jdtls/eclipse-jdt-core-incubator/blob/dom-with-javac/org.eclipse.jdt.core.javac/META-INF/p2.inf

This opens a bunch of jdk internals that the jdt-javac work uses,
and also sets the flags to enable the javac-based search.

Signed-off-by: David Thompson <[email protected]>
@datho7561 datho7561 force-pushed the update-flags-for-jdt-javac branch from 9bc69a2 to ebbc428 Compare October 8, 2025 18:31
@mickaelistria
Copy link
Contributor

Is completion flag -DICompletionEngineProvider=org.eclipse.jdt.core.dom.DOMCompletionEngineProvider intentionally skipped?

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