Skip to content

Conversation

@mdeinum
Copy link
Contributor

@mdeinum mdeinum commented Nov 18, 2020

There some locations which could benefit from not using a
toCharArray on a String, but rather use the charAt method from
the String itself. This to prevent an additional copy of the
char[] being created.

There some locations which could benefit from not using a
toCharArray on a String, but rather use the charAt method from
the String itself. This to prevent an additional copy of the
char[] being created.
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Nov 18, 2020
@mdeinum
Copy link
Contributor Author

mdeinum commented Nov 18, 2020

Whilst optimizing the SimpleHttpCodeStatusMapper and SimpleStatusAggregator I noticed they both share the exact same getUniformCode method. Maybe that should be moved to a shared class or to the Status class as to avoid duplication?

@philwebb philwebb added type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged labels Nov 18, 2020
@philwebb philwebb added this to the 2.5.x milestone Nov 18, 2020
@snicoll snicoll modified the milestones: 2.5.x, 2.5.0-M1 Dec 21, 2020
@snicoll snicoll self-assigned this Dec 21, 2020
snicoll pushed a commit that referenced this pull request Dec 21, 2020
snicoll added a commit that referenced this pull request Dec 21, 2020
@snicoll snicoll closed this in bb78b6a Dec 21, 2020
@snicoll
Copy link
Member

snicoll commented Dec 21, 2020

@mdeinum thanks. As for the duplication, it's fine as it is.

@mdeinum mdeinum deleted the reduce_overhead_of_tochararray branch April 29, 2021 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: enhancement A general enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants