RFR 8146625: OverloadedDynamicMethod has unused ClassLoader field that can be removed
Sundararajan Athijegannathan
sundararajan.athijegannathan at oracle.com
Thu Jan 7 16:13:50 UTC 2016
Thanks for the suggestion & review.
I'm going ahead with this updated webrev ->
http://cr.openjdk.java.net/~sundar/8146625/webrev.01/
-Sundar
On 1/7/2016 9:18 PM, Attila Szegedi wrote:
> +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