Skip to content

[GR-42112] Missing registration errors for reflection calls #6139

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Mar 13, 2023
Merged

[GR-42112] Missing registration errors for reflection calls #6139

merged 1 commit into from
Mar 13, 2023

Conversation

graalvmbot
Copy link
Collaborator

No description provided.

@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Mar 7, 2023
@loicottet
Copy link
Member

Introduces MissingReflectionRegistrationError when it is impossible to tell whether a reflective query should fail because the queried element has not been registered at build-time, or because it doesn't exist. The PR also adds support for Class.getNestMembers and Class.getSigners.
The changes are hidden behind the -H:+ThrowMissingRegistrationErrors flag for now, with the objective of enabling this flag by default in the future and eventually getting rid of it.

@loicottet loicottet self-requested a review March 7, 2023 12:59
@graalvmbot graalvmbot force-pushed the lottet/missing-method branch from 3a54ab4 to 737af06 Compare March 7, 2023 13:07
@bclozel bclozel mentioned this pull request Mar 7, 2023
Closed
@graalvmbot graalvmbot force-pushed the lottet/missing-method branch 11 times, most recently from f8916b4 to dd0cb60 Compare March 10, 2023 16:07
@graalvmbot graalvmbot force-pushed the lottet/missing-method branch from dd0cb60 to 5afb1b5 Compare March 12, 2023 09:28
@graalvmbot graalvmbot merged commit 3a73004 into master Mar 13, 2023
@graalvmbot graalvmbot deleted the lottet/missing-method branch March 13, 2023 00:18
@bclozel bclozel mentioned this pull request Sep 13, 2023
Closed
@fniephaus fniephaus mentioned this pull request May 27, 2024
Open
@zakkak zakkak mentioned this pull request Jul 19, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@loicottet loicottet Awaiting requested review from loicottet

Assignees
No one assigned
Labels
OCA Verified All contributors have signed the Oracle Contributor Agreement.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@graalvmbot @loicottet