From naoto.sato at oracle.com Wed May 6 16:50:16 2015 From: naoto.sato at oracle.com (Naoto Sato) Date: Wed, 06 May 2015 09:50:16 -0700 Subject: [9] RFR: 8079186: Add 'localeServiceProvider' target in the class description of RuntimePermission Message-ID: <554A4648.7050708@oracle.com> Hi Sean, Can you please review the change to RuntimePermission's class document? This is a follow-on fix to the prior fix to 8075545. Bug: https://bugs.openjdk.java.net/browse/JDK-8079186 Fix: http://cr.openjdk.java.net/~naoto/8079186/webrev.00/ Naoto From sean.mullan at oracle.com Thu May 7 14:35:42 2015 From: sean.mullan at oracle.com (Sean Mullan) Date: Thu, 07 May 2015 10:35:42 -0400 Subject: [9] RFR: 8079186: Add 'localeServiceProvider' target in the class description of RuntimePermission In-Reply-To: <554A4648.7050708@oracle.com> References: <554A4648.7050708@oracle.com> Message-ID: <554B783E.6000504@oracle.com> Looks fine. Don't forget to add a noreg-doc label to the bug. --Sean On 05/06/2015 12:50 PM, Naoto Sato wrote: > Hi Sean, > > Can you please review the change to RuntimePermission's class document? > This is a follow-on fix to the prior fix to 8075545. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8079186 > Fix: http://cr.openjdk.java.net/~naoto/8079186/webrev.00/ > > Naoto From mark.reinhold at oracle.com Thu May 14 20:46:41 2015 From: mark.reinhold at oracle.com (mark.reinhold at oracle.com) Date: Thu, 14 May 2015 13:46:41 -0700 (PDT) Subject: JEP 252: Use CLDR Locale Data by Default Message-ID: <20150514204641.2376C62C8B@eggemoggin.niobe.net> New JEP Candidate: http://openjdk.java.net/jeps/252 - Mark From mandy.chung at oracle.com Mon May 18 22:45:17 2015 From: mandy.chung at oracle.com (Mandy Chung) Date: Mon, 18 May 2015 15:45:17 -0700 Subject: Review Request for 8074431: Remove native2ascii tool Message-ID: <555A6B7D.6060809@oracle.com> This patch removes native2ascii command-line tool from JDK 9 as proposed in March [1]. Webrev at: http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8074431/webrev.00/ Jon - A langtools test is updated to use a new Native2Ascii.java helper class instead. Existing code that calls sun.tools.native2ascii.Main can copy the implementation or Native2Ascii.java. thanks Mandy [1] http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-March/032634.html From Alan.Bateman at oracle.com Tue May 19 07:02:48 2015 From: Alan.Bateman at oracle.com (Alan Bateman) Date: Tue, 19 May 2015 08:02:48 +0100 Subject: Review Request for 8074431: Remove native2ascii tool In-Reply-To: <555A6B7D.6060809@oracle.com> References: <555A6B7D.6060809@oracle.com> Message-ID: <555AE018.5050109@oracle.com> On 18/05/2015 23:45, Mandy Chung wrote: > This patch removes native2ascii command-line tool from JDK 9 as > proposed in March [1]. Looks good. -Alan From erik.joelsson at oracle.com Tue May 19 08:18:49 2015 From: erik.joelsson at oracle.com (Erik Joelsson) Date: Tue, 19 May 2015 10:18:49 +0200 Subject: Review Request for 8074431: Remove native2ascii tool In-Reply-To: <555A6B7D.6060809@oracle.com> References: <555A6B7D.6060809@oracle.com> Message-ID: <555AF1E9.9040302@oracle.com> Looks good to me. /Erik On 2015-05-19 00:45, Mandy Chung wrote: > This patch removes native2ascii command-line tool from JDK 9 as > proposed in March [1]. > > Webrev at: > http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8074431/webrev.00/ > > Jon - A langtools test is updated to use a new Native2Ascii.java > helper class instead. > > Existing code that calls sun.tools.native2ascii.Main can copy the > implementation or Native2Ascii.java. > > thanks > Mandy > [1] > http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-March/032634.html > From jonathan.gibbons at oracle.com Mon May 18 23:02:29 2015 From: jonathan.gibbons at oracle.com (Jonathan Gibbons) Date: Mon, 18 May 2015 16:02:29 -0700 Subject: Review Request for 8074431: Remove native2ascii tool In-Reply-To: <555A6B7D.6060809@oracle.com> References: <555A6B7D.6060809@oracle.com> Message-ID: <555A6F85.9090409@oracle.com> On 05/18/2015 03:45 PM, Mandy Chung wrote: > This patch removes native2ascii command-line tool from JDK 9 as > proposed in March [1]. > > Webrev at: > http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8074431/webrev.00/ > > Jon - A langtools test is updated to use a new Native2Ascii.java > helper class instead. > > Existing code that calls sun.tools.native2ascii.Main can copy the > implementation or Native2Ascii.java. > > thanks > Mandy > [1] > http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-March/032634.html > LangTools changes OK by me. -- Jon From xueming.shen at oracle.com Sun May 24 00:26:40 2015 From: xueming.shen at oracle.com (Xueming Shen) Date: Sat, 23 May 2015 17:26:40 -0700 Subject: RFR JDK-8042125: Japanese character converters incompatible between Java 7 and Java 8 Message-ID: <55611AC0.8090803@oracle.com> Hi Please help the change for 8042125 issue: https://bugs.openjdk.java.net/browse/JDK-8042125 webrev: http://cr.openjdk.java.net/~sherman/8042125 It's a regression caused by the changes of JDK-6653797. The direct triggers are (1) the .c2b mapping table for ms932/0208is missing (regardless the comment in JIS_X_0208_MS932.map says we need one) (2) mapping entry for those non-roundtrip code points are mistakenly commented out in jis_x_0212_solaris.map and jis_x_0208_ms932.map. thanks! -Sherman From masayoshi.okutsu at oracle.com Mon May 25 09:38:12 2015 From: masayoshi.okutsu at oracle.com (Masayoshi Okutsu) Date: Mon, 25 May 2015 18:38:12 +0900 Subject: RFR JDK-8042125: Japanese character converters incompatible between Java 7 and Java 8 In-Reply-To: <55611AC0.8090803@oracle.com> References: <55611AC0.8090803@oracle.com> Message-ID: <5562ED84.3070204@oracle.com> Looks good to me. Masayoshi On 5/24/2015 9:26 AM, Xueming Shen wrote: > Hi > > Please help the change for 8042125 > > issue: https://bugs.openjdk.java.net/browse/JDK-8042125 > webrev: http://cr.openjdk.java.net/~sherman/8042125 > > It's a regression caused by the changes of JDK-6653797. The direct > triggers are > (1) the .c2b mapping table for ms932/0208is missing (regardless the > comment in > JIS_X_0208_MS932.map says we need one) > (2) mapping entry for those non-roundtrip code points are mistakenly > commented > out in jis_x_0212_solaris.map and jis_x_0208_ms932.map. > > thanks! > -Sherman > > >