On 2016-08-09 16:27, Paul Sandoz wrote:
On 9 Aug 2016, at 05:04, Claes Redestad <claes.redes...@oracle.com> wrote:
Hi,
please review this set of small changes to reduce the number of classes defined
and pulled in when initializing java.lang.invoke
Webrev: http://cr.openjdk.java.net/~redestad/8163370/webrev/
Bug: https://bugs.openjdk.java.net/browse/JDK-8163370
- Using an EnumMap<Wrapper, MethodHandle> in sun.misc.invoke.ValueConversions
pulls in 5 EnumMap-related classes, and
- Using result.addAll(Arrays.asList(buf).subList(0, bufCount)) rather than a
simple for-loop pulls in 5 classes
- A few private enum classes can profitably be replaced with alternative logic,
removing 4 classes
I browsed this quite quickly, looks good.
Thanks!
Glad you added the assert in Wrapper.java.
I'd not sleep well without that assert. :-)
/Claes
