JDK 13 RFR of core libs portions of JDK-8205626: Start of release updates for JDK 13

Andrew Luo andrewluotechnologies at outlook.com
Sat Dec 8 02:18:12 UTC 2018


Not a reviewer - looks good anyways - however, I would if some of those constants be refactored to fewer locations?

For example, I see many copies of:

@SupportedSourceVersion(SourceVersion.RELEASE_13)

Perhaps if you declare SourceVersion as:

RELEASE_12,
RELEASE_13,
CURRENT_RELEASE = RELEASE_13;

Then you could use:

@SupportedSourceVersion(SourceVersion.CURRENT_RELEASE)

Thanks,

-Andrew

-----Original Message-----
From: core-libs-dev <core-libs-dev-bounces at openjdk.java.net> On Behalf Of joe darcy
Sent: Friday, December 7, 2018 9:52 AM
To: Alan Bateman <Alan.Bateman at oracle.com>; Core-Libs-Dev <core-libs-dev at openjdk.java.net>
Subject: Re: JDK 13 RFR of core libs portions of JDK-8205626: Start of release updates for JDK 13

Hi Alan,

On 12/7/2018 12:16 AM, Alan Bateman wrote:
> On 07/12/2018 02:31, Joseph D. Darcy wrote:
>> Hello,
>>
>> With the start of JDK 13 around the corner, please review the core 
>> libs portions of:
>>
>>     JDK-8205626: Start of release updates for JDK 13
>>     http://cr.openjdk.java.net/~darcy/jdk13-fork.2
>>
>>
[snip]

> Looks okay (and the same as what we reviewed on build-dev yesterday?).
>
Yes, same changes as on build-dev (other than the update of several more TEST.ROOT files to require jtreg 4.2 b13 rather than b12.)

I wanted to have a bit more of the testing complete before sending the core libs portion out for the review.

Thanks,

-Joe



More information about the core-libs-dev mailing list