Suggestion: Make KiwiMaps.newTreeMap return a SortedMap instead of just Map

[chrisrohr]

There are a number of instances where libraries need SortedMap instances and using newTreeMap returns a Map which causes compile time issues. Since TreeMap implements SortedMap which implements Map, this should work in most cases.

[sleberknight]

Odd...I just looked at the original (internal) KiwiMaps and that method returns Mapnot SortedMap, so how did it work internally in the places where we now have been forced to cast to SortedMap?

[sleberknight]

So I can remember, we first encountered the need to cast to a SortedMap in dropwizard-service-utilities in the HttpConnectionsHealthCheckTest

[sleberknight]

Odd...I just looked at the original (internal) KiwiMaps and that method returns Mapnot SortedMap, so how did it work internally in the places where we now have been forced to cast to SortedMap?

I was wrong...it does return SortedMap, which is why it works...

[sleberknight]

Signature should be:

public static <K extends Comparable, V> SortedMap<K, V> newTreeMap(Object... items)