Re: RFR: JDK11U JDK-8206120, JDK-8211398, JDK-8218915

2019-02-21 Thread Naoto Sato
om>; core-libs-dev <mailto:core-libs-dev@openjdk.java.net>;jdk-updates-...@openjdk.java.net <mailto:jdk-updates-...@openjdk.java.net> Subject: Re: RFR: JDK11U JDK-8206120, JDK-8211398, JDK-8218915 Hi Deepak, I see the following comment is not addressed yet: &g

Re: RFR: JDK11U JDK-8206120, JDK-8211398, JDK-8218915

2019-02-21 Thread Seán Coffey
Coffey *Sent:* Thursday, February 21, 2019 2:11 PM *To:* Deepak Kejriwal ; Naoto Sato ; core-libs-dev ; jdk-updates-...@openjdk.java.net *Subject:* Re: RFR: JDK11U JDK-8206120, JDK-8211398, JDK-8218915 Deepak, this exception message in new test still needs correction: 166

RE: RFR: JDK11U JDK-8206120, JDK-8211398, JDK-8218915

2019-02-21 Thread Deepak Kejriwal
; core-libs-dev ; jdk-updates-...@openjdk.java.net Subject: Re: RFR: JDK11U JDK-8206120, JDK-8211398, JDK-8218915   Deepak, this exception message in new test still needs correction: 166 "Character.isLetter(int) failed for codepoint &

Re: RFR: JDK11U JDK-8206120, JDK-8211398, JDK-8218915

2019-02-21 Thread Seán Coffey
for review. Regards, Deepak -Original Message- From: Naoto Sato Sent: Wednesday, February 20, 2019 11:06 PM To: Deepak Kejriwal ; Sean Coffey ; core-libs-dev ; jdk-updates-...@openjdk.java.net Subject: Re: RFR: JDK11U JDK-8206120, JDK-8211398, JDK-8218915 Hi Deepak, I see the followi

RE: RFR: JDK11U JDK-8206120, JDK-8211398, JDK-8218915

2019-02-21 Thread Deepak Kejriwal
Kejriwal ; Sean Coffey ; core-libs-dev ; jdk-updates-...@openjdk.java.net Subject: Re: RFR: JDK11U JDK-8206120, JDK-8211398, JDK-8218915 Hi Deepak, I see the following comment is not addressed yet: > - Line 163,198: Exception messages are incorrect. they are for > isJavaIdentifie

Re: RFR: JDK11U JDK-8206120, JDK-8211398, JDK-8218915

2019-02-20 Thread naoto . sato
RFR: JDK11U JDK-8206120, JDK-8211398, JDK-8218915 Hi Deepak, Here are my comments to the webrev (other than what Sean pointed out): TestIsJavaIdentifierMethods.java - @summary: "testIsJavaLetter" -> "isJavaLetter", "testIsJavaLetterOrDigit" -> "isJavaLet

RE: RFR: JDK11U JDK-8206120, JDK-8211398, JDK-8218915

2019-02-20 Thread Deepak Kejriwal
To: Deepak Kejriwal ; core-libs-dev ; jdk-updates-...@openjdk.java.net Subject: Re: RFR: JDK11U JDK-8206120, JDK-8211398, JDK-8218915 Hi Deepak, Here are my comments to the webrev (other than what Sean pointed out): TestIsJavaIdentifierMethods.java - @summary: "testIsJavaL

Re: RFR: JDK11U JDK-8206120, JDK-8211398, JDK-8218915

2019-02-19 Thread Naoto Sato
racter is a connecting punctuation character (such as '_'). - Line 163,198: Exception messages are incorrect. they are for isJavaIdentifierStart(). Naoto On 2/19/19 6:15 AM, Deepak Kejriwal wrote: Correcting typo for release. From: Deepak Kejriwal Sent: Tuesday, February 19, 2019 7:42 P

Re: RFR: JDK11U JDK-8206120, JDK-8211398, JDK-8218915

2019-02-19 Thread Seán Coffey
t: Tuesday, February 19, 2019 7:42 PM To: 'core-libs-dev' ; 'jdk-updates-...@openjdk.java.net' Subject: RFR: JDK11U JDK-8206120, JDK-8211398, JDK-8218915 Hi All, Please review the backport of the following bug fixes to jdk11u-dev: HYPERLINK "https://bugs.openjdk.java.net/browse/JDK-8206

Re: RFR: JDK11U JDK-8206120, JDK-8211398, JDK-8218915

2019-02-19 Thread Alan Bateman
On 19/02/2019 14:25, Aleksey Shipilev wrote: Hi Deepak, Please follow the jdk-updates approval process: https://openjdk.java.net/projects/jdk-updates/approval.html You need to put appropriate tags at the issues, explain why they are needed, how the changes apply (if changes do not apply

Re: RFR: JDK11U JDK-8206120, JDK-8211398, JDK-8218915

2019-02-19 Thread Aleksey Shipilev
ev' ; > 'jdk-updates-...@openjdk.java.net' > Subject: RFR: JDK11U JDK-8206120, JDK-8211398, JDK-8218915 > > > > Hi All, > > Please review the backport of the following bug fixes to jdk11u-dev: > > HYPERLINK "https://bugs.openjdk.java.net/browse/JDK-8206120"JDK

RE: RFR: JDK11U JDK-8206120, JDK-8211398, JDK-8218915

2019-02-19 Thread Deepak Kejriwal
Correcting typo for release. From: Deepak Kejriwal Sent: Tuesday, February 19, 2019 7:42 PM To: 'core-libs-dev' ; 'jdk-updates-...@openjdk.java.net' Subject: RFR: JDK11U JDK-8206120, JDK-8211398, JDK-8218915 Hi All, Please review the backport of the following bug fixes to jdk11u-dev

RFR: JDK11U JDK-8206120, JDK-8211398, JDK-8218915

2019-02-19 Thread Deepak Kejriwal
Hi All, Please review the backport of the following bug fixes to jdk8u-dev: HYPERLINK "https://bugs.openjdk.java.net/browse/JDK-8206120"JDK-8206120 : Add test cases for lenient Japanese era parsing HYPERLINK "https://bugs.openjdk.java.net/browse/JDK-8211398"JDK-8211398 : Square character