I've found a few places, that I happened to bump into while debugging, that do stuff like:
<snip>
throw new IllegalArgumentException(String
.format("Cannot create default factory for class: %s",
String.valueOf(clazz)));
</snip>
This is complexly pointless use of String.valueOf(), as this is exactly what
the "%s" token will do. So its just wasting a method call, inflating the
byte-code and cluttering up the source.
--jason
