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
