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

Use jdk22 GA instead of EA #1

Open
wants to merge 143 commits into
base: jdk22
Choose a base branch
from
Open

Conversation

rspilker
Copy link

I've already done the docker provisioning update, that should fix the broken check

rspilker and others added 30 commits March 18, 2024 00:03
This code was never used because StringLiteralConcatenation and
ExtendedStringLiteral extend StringLiterals and get handled a few lines
above. We might need to add it back and fix it if we want to copy
annotations like @annotation("a" + "b") without transforming them to
@annotation("ab")
…s correct, it is the tests that are wrong:

 Useful for small refactors that cause big test file changes.
…e lost. 15 years and nobody found this bug - not worth an issue or a changelog.
…-cases now have `@Generated` annotations where before they did not.
rzwitserloot and others added 30 commits October 17, 2024 23:51
…ever so slightly; timestamps of zip entries is now preserved.
Update annotation value index
- closeQuietly is a pet peeve of mine; it is an abomination that must never be.
- All actual resources are closed; all filters aren't because there is no need.
- the underlying problem (of 'flushing' zips) is addressed.
This reverts commit 04c9755.

The nature of the fix is to simply scan the TypeMirror for 'lombok shenanigans'; if it finds them, the type is not complete; if there are none, the type is.

This fundamentally does not work - lombok shenanigans may remain even if lombok is done (`@lombok.NonNull` for example), and lombok shenanigans don't just appear as annotations on the type; they can appear in many forms: Annotations on local var decls, or even method calls to `Lombok.safeDeNull` or whatnot. safeDenull is made up, but we might add it someday. `@Getter` on a field isn't though, and suffers from the same problem.
[ci] Switch to Ubuntu 24.04 (ships with ant 1.10.14)
fix: Don't alter the AST node positions of original code.
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.