RFR 8146625: OverloadedDynamicMethod has unused ClassLoader field that can be removed

Attila Szegedi szegedia at gmail.com
Thu Jan 7 15:48:27 UTC 2016


+1; maybe move the “methods” field initialization to its declaration (so you can also use <> for the component type):

private final LinkedList<SingleDynamicMethod> methods = new LinkedList<>();

Attila.

> On Jan 7, 2016, at 4:16 PM, Sundararajan Athijegannathan <sundararajan.athijegannathan at oracle.com> wrote:
> 
> Please review http://cr.openjdk.java.net/~sundar/8146625/ for https://bugs.openjdk.java.net/browse/JDK-8146625
> 
> Thanks,
> -Sundar




More information about the nashorn-dev mailing list