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

Simplify ConstructorResolver: do not sort intermediate array #23969

Merged
merged 1 commit into from Nov 11, 2019

Conversation

@stsypanov
Copy link
Contributor

stsypanov commented Nov 11, 2019

There's no need to create an intermediate array of Method candidates just to sort it. Intead we can directly sort existing ArrayList which costs us the same as there's array under the hood.

@jhoeller jhoeller self-assigned this Nov 11, 2019
@jhoeller jhoeller added this to the 5.2.2 milestone Nov 11, 2019
@jhoeller jhoeller merged commit 92efe95 into spring-projects:master Nov 11, 2019
1 check passed
1 check passed
ci/pivotal-cla Thank you for signing the Contributor License Agreement!
Details
@stsypanov stsypanov deleted the stsypanov:simplify-const-resolver branch Nov 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.