Review request for JDK-8138906

Srinivas Dama srinivas.dama at
Tue Mar 1 12:00:37 UTC 2016

Hi Michael,


Thank you .


Here is the latest webrev with all modifications.


Bug :
Webrev :




From: Michael Haupt 
Sent: Tuesday, March 01, 2016 3:16 PM
To: Srinivas Dama
Cc: nashorn-dev at
Subject: Re: Review request for JDK-8138906


Hi Srinivas,


Am 29.02.2016 um 17:28 schrieb Srinivas Dama <HYPERLINK "mailto:srinivas.dama at"srinivas.dama at>:

Please review :
Bug :
Webrev :


I've verified this works on Windows. Lower-case thumbs up, with these remarks:


== JDK-util.js ==


Lines 64/65: typos, please use

"Unix cygpath implementation.

Supports only two outputs, windows (C:\dir\) and mixed (C:/dir/)."


== JDK-8087292.js ==


Copyright: convention requires a space after the comma in "2015,2016", and a comma after "2016", i.e., please use "2015, 2016, ".


As a matter of personal preference, how about using semicolons consistently, i.e., either at the ends of all lines, or never?

I have added some utility functions(can be added more) which can be used in all nashorn test case to make them work
Independent of underlying shell.(removed dependency on shell commands like : which)


These look useful and should be adopted by more tests (that's a future RFE).







HYPERLINK "" \nOracle
Dr. Michael Haupt | Principal Member of Technical Staff
Phone: +49 331 200 7277 | Fax: +49 331 200 7561
Oracle Java Platform Group | LangTools Team | Nashorn
Oracle Deutschland B.V. & Co. KG | Schiffbauergasse 14 | 14467 Potsdam, Germany


ORACLE Deutschland B.V. & Co. KG | Hauptverwaltung: Riesstraße 25, D-80992 München
Registergericht: Amtsgericht München, HRA 95603

Komplementärin: ORACLE Deutschland Verwaltung B.V. | Hertogswetering 163/167, 3543 AS Utrecht, Niederlande
Handelsregister der Handelskammer Midden-Nederland, Nr. 30143697
Geschäftsführer: Alexander van der Ven, Jan Schultheiss, Val Maher

HYPERLINK "" \nGreen Oracle

Oracle is committed to developing practices and products that help protect the environment


More information about the nashorn-dev mailing list