Skip to content

Backport "Fix Java record problems (#19578) and (#19386)" to LTS - #20890

Merged
WojciechMazur merged 6 commits into
lts-3.3from
lts-19583
Jul 1, 2024
Merged

Backport "Fix Java record problems (#19578) and (#19386)" to LTS#20890
WojciechMazur merged 6 commits into
lts-3.3from
lts-19583

Conversation

@WojciechMazur

Copy link
Copy Markdown
Contributor

Backports #19583 to the LTS branch.

PR submitted by the release tooling.
[skip ci]

yilinwei added 6 commits June 30, 2024 23:59
This fixes a whole host of subtle issues.

 - The type parameter was not stamped correctly on the constructor
   causing the original error
 - The parsed record was not stamped with `JavaDefined`, which meant
   the duplicate constructors in the case of overrides were not
   removed.

[Cherry-picked f7eb589]
We change the proxy method to take in a single argument. This also
exposed a second issue where the overriden methods were not
invalidated correctly in the `Namer`.

[Cherry-picked 47511ae]
Base automatically changed from lts-19645 to lts-3.3 July 1, 2024 08:42
@WojciechMazur

Copy link
Copy Markdown
Contributor Author

No regressions detected in the community build up to lts-19605.

Reference

@WojciechMazur
WojciechMazur merged commit 95b8e13 into lts-3.3 Jul 1, 2024
@WojciechMazur
WojciechMazur deleted the lts-19583 branch July 1, 2024 08:42
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