Skip to content

Use Method#isDefault to find default methods in interfaces #31197

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
Closed
wants to merge 1 commit into from
Closed

Use Method#isDefault to find default methods in interfaces #31197

wants to merge 1 commit into from

Conversation

refeccd
Copy link
Contributor

@refeccd refeccd commented Sep 10, 2023 •
edited by sbrannen
Loading

Use Method#isDefault to replace the logic in ReflectionUtils#findConcreteMethodsOnInterfaces.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Sep 10, 2023
@sbrannen sbrannen mentioned this pull request Sep 10, 2023
Closed
@sbrannen sbrannen self-assigned this Sep 10, 2023
@sbrannen sbrannen added in: core Issues in core modules (aop, beans, core, context, expression) type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Sep 10, 2023
@sbrannen sbrannen added this to the 6.1.0-RC1 milestone Sep 10, 2023
@sbrannen sbrannen changed the title Use jdk api when getting the default method of the interface Use Method#isDefault to find default methods in interfaces Sep 10, 2023
@sbrannen sbrannen closed this in 9728b8c Sep 10, 2023
@sbrannen
Copy link
Member

This has been merged into main.

Thanks

@refeccd refeccd deleted the patch-1 branch September 10, 2023 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@sbrannen sbrannen

Labels
in: core Issues in core modules (aop, beans, core, context, expression) type: enhancement A general enhancement
Projects
None yet
Milestone
6.1.0-M5
Development

Successfully merging this pull request may close these issues.
3 participants
@refeccd @sbrannen @spring-projects-issues