Will fix that, thanks for the reviews!. I was told by Amy,
meanwhile this test has been added to ProblemList.txt so
I will have to remove that line as well.
Thanks
Kumar
On 05/18/2016 05:20 PM, Kumar Srinivasan wrote:
The second part is fixing up the test, which does not read the
process streams correctly and causes a test hang, simplified
the test with time tested utilities.
http://cr.openjdk.java.net/~ksrini/8156807/webrev.00/
A very minor snag in Constants.java:
41 /*
42 Java Class Version numbers history
43 1.0 to 1.3.X 45,3
44 1.4 to 1.4.X 46,0
45 1.5 to 1.5.X 49,0
46 1.6 to 1.6.X 50,0
47 1.7 to 1.7.X 51,0
48 1.8 to 1.8.X 52,0
49 1.9 to 1.9.X 53.0
50 */
Should be "53,0"? Notice the comma. Fix as you deem appropriate, no need
for re-review.
Otherwise looks good.
Thanks,
-Aleksey
