RFR: 8156807: Pack200 must support v53.0 class files

Kumar Srinivasan kumar.x.srinivasan at oracle.com
Wed May 18 16:36:21 UTC 2016


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
>




More information about the core-libs-dev mailing list