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.
Comment From: mdeinum
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?
Comment From: snicoll
@mdeinum thanks. As for the duplication, it's fine as it is.