RFR(S) 8158338: Nashorn's ScriptLoader split delegation has to adjusted
Sundararajan Athijegannathan
sundararajan.athijegannathan at oracle.com
Wed Jun 1 07:29:12 UTC 2016
Hi,
Thanks for the review.
We have an existing test that depends on "split" delegation implemented
in the ScriptLoader.
http://hg.openjdk.java.net/jdk9/dev/nashorn/file/7fb2bf00347b/test/script/basic/JDK-8024619.js
The current code adjustment is about which loader is "parent" and which
one "side" delegatee. Previously, parent was Context's appLoader and
side-delegatee was the structure loader.
Now, it is other way around. But, the behavior seen from scripts should
remain same in either case. The aforementioned test (along with other
nashorn tests) passes with the adjusted
delegation setup as well.
-Sundar
On 6/1/2016 12:49 PM, Marcus Lagergren wrote:
> This looks good. Is it possible to test it?
>
> /M
>
>> On 01 Jun 2016, at 08:46, Sundararajan Athijegannathan <sundararajan.athijegannathan at oracle.com> wrote:
>>
>> Please review http://cr.openjdk.java.net/~sundar/8158338/webrev.00/ for
>> https://bugs.openjdk.java.net/browse/JDK-8158338
>>
>> Thanks,
>>
>> -Sundar
>>
More information about the nashorn-dev
mailing list