RFR: 8184241(Fix nashorn/samples/filebrowser.js)
Hannes Wallnöfer
hannes.wallnoefer at oracle.com
Wed Jul 19 09:21:13 UTC 2017
Hi Srini,
The fix looks good, but what for is the arguments handling in the test?
Also, I would prefer to use our own base class (constructor must invoke overridable method, what if TreeItem is refactored in the future?)
Hannes
> Am 19.07.2017 um 10:45 schrieb Srinivas Dama <srinivas.dama at oracle.com>:
>
> Hi,
>
> Please review http://cr.openjdk.java.net/~sdama/8184241/webrev.00/ for https://bugs.openjdk.java.net/browse/JDK-8184241.
>
> Regards,
> Srinivas
More information about the nashorn-dev
mailing list