On Fri, 27 May 2022 18:40:32 GMT, XenoAmess wrote:
>> as title.
>
> XenoAmess has updated the pull request incrementally with one additional
> commit since the last revision:
>
> do it as naotoj said
Reviewed i18n-related changes and they look good. One minor suggestion in
`Calendar`, but t
On Wed, 18 May 2022 13:27:24 GMT, Alexey Ivanov wrote:
> Replaces usages of articles that follow each other in all combinations:
> a/the, an?/an?, the/the…
>
> Also, I fixed a couple of spelling mistakes.
LGTM for i18n changes.
-
Marked as reviewed by naoto (Reviewer).
PR: https
On Tue, 10 May 2022 21:32:10 GMT, Roger Riggs wrote:
> PR#8599 8244681: proposes to add compiler warnings for possible lossy
> conversions
> From the CSR:
>
> "If the type of the right-hand operand of a compound assignment is not
> assignment compatible with the type of the variable, a cast is
On Tue, 19 Apr 2022 16:50:12 GMT, Magnus Ihse Bursie wrote:
>> I ran `codespell` on the `src/java.base` directory, and accepted those
>> changes where it indeed discovered real typos.
>>
>> (Due to false positives this can unfortunately not be run automatically)
>>
>> The majority of fixes ar
On Thu, 14 Apr 2022 18:32:03 GMT, Naoto Sato wrote:
>>> Good point! Filed an issue: https://bugs.openjdk.java.net/browse/JDK-8284856
>>
>> @stuart-marks @naotoj I can help solve JDK-8284856 after this pr. But
>> usually we only solve 1 issue in 1 pr, so I thin
On Thu, 14 Apr 2022 19:07:09 GMT, Magnus Ihse Bursie wrote:
> I ran `codespell` on the `src/java.base` directory, and accepted those
> changes where it indeed discovered real typos.
>
> (Due to false positives this can unfortunately not be run automatically)
>
> The majority of fixes are in c
On Thu, 14 Apr 2022 17:06:53 GMT, XenoAmess wrote:
>> Good point! Filed an issue: https://bugs.openjdk.java.net/browse/JDK-8284856
>
>> Good point! Filed an issue: https://bugs.openjdk.java.net/browse/JDK-8284856
>
> @stuart-marks @naotoj I can help solve JDK-8284856 after this pr. But usually
On Wed, 13 Apr 2022 23:48:06 GMT, Stuart Marks wrote:
> but I suspect the cleanup may simply be removing them entirely.
+1 for removing it.
-
PR: https://git.openjdk.java.net/jdk/pull/7928
On Wed, 13 Apr 2022 22:40:38 GMT, Stuart Marks wrote:
>> XenoAmess has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> update LastModified
>
> src/java.base/share/classes/java/lang/Character.java line 8574:
>
>> 8572: private static
On Wed, 13 Apr 2022 16:29:11 GMT, XenoAmess wrote:
>> 8186958: Need method to create pre-sized HashMap
>
> XenoAmess has updated the pull request incrementally with one additional
> commit since the last revision:
>
> revert changes in:
> src/java.desktop
> src/java.management
> src/jdk
On Wed, 30 Mar 2022 00:43:49 GMT, Alisen Chung wrote:
> redo of 8280400
I believe
`src/jdk.localedata/share/classes/sun/util/resources/ext/CurrencyNames_zh_CN.properties`
and `test/jdk/sun/text/resources/LocaleData` have to be adjusted according to
the l10n changes.
-
PR: https:
On Mon, 28 Mar 2022 21:20:00 GMT, Alisen Chung wrote:
> This reverts commit c0aecd15ae8d7abf37901f785fccaff2317c3b23.
LGTM
-
Marked as reviewed by naoto (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/8005
On Wed, 16 Mar 2022 18:31:55 GMT, Alisen Chung wrote:
>> msg drop for jdk19, Mar 9, 2022
>
> Alisen Chung has updated the pull request incrementally with one additional
> commit since the last revision:
>
> removed incorrect translation of compiler configuration file
LGTM. Thanks for the cha
On Mon, 14 Mar 2022 20:39:46 GMT, Alisen Chung wrote:
>> msg drop for jdk19, Mar 9, 2022
>
> Alisen Chung has updated the pull request incrementally with one additional
> commit since the last revision:
>
> removed repeated lines from ROOT bundle in CurrencyNames
src/jdk.compiler/share/class
On Thu, 10 Mar 2022 17:55:44 GMT, Alisen Chung wrote:
>> msg drop for jdk19, Mar 9, 2022
>
> Alisen Chung has updated the pull request incrementally with one additional
> commit since the last revision:
>
> moved CurrencyNames changes to jdk.localedata
src/jdk.localedata/share/classes/sun/ut
On Wed, 9 Mar 2022 21:09:30 GMT, Alisen Chung wrote:
> msg drop for jdk19, Mar 9, 2022
IIRC, localized resource files should have the same copyright year as the base
English one. That's what I was told by the l10n engineer when I had the same
comment.
-
PR: https://git.openjdk.ja
On Wed, 9 Mar 2022 21:09:30 GMT, Alisen Chung wrote:
> msg drop for jdk19, Mar 9, 2022
`src/java.base/share/classes/sun/util/resources/CurrencyNames_de.properties`
`src/java.base/share/classes/sun/util/resources/CurrencyNames_ja.properties`
`src/java.base/share/classes/sun/util/resources/Currenc
On Wed, 16 Feb 2022 21:27:40 GMT, Daniel Fuchs wrote:
> Please find here a changeset to ProblemList
> sun/net/www/protocol/https/HttpsURLConnection/B6216082.java until JDK-8282017
> is fixed
Marked as reviewed by naoto (Reviewer).
-
PR: https://git.openjdk.java.net/jdk/pull/7505
On Wed, 1 Dec 2021 14:47:54 GMT, Roger Riggs wrote:
> Compilation warnings of the test library introduce noise in test output and
> should be addressed or suppressed.
> Changes include:
> - SuppressWarnings("deprecation") and SuppressWarnings("removal")
> - Adding type parameters to Raw types
On Tue, 12 Oct 2021 20:39:13 GMT, Andrey Turbanov wrote:
>> StringBuffer is a legacy synchronized class. There are more modern
>> alternatives which perform better:
>> 1. Plain String concatenation should be preferred
>> 2. StringBuilder is a direct replacement to StringBuffer which generally
>
On Mon, 11 Oct 2021 21:05:46 GMT, Andrey Turbanov wrote:
>> src/java.base/share/classes/java/lang/Character.java line 123:
>>
>>> 121: * than U+ are called supplementary characters. The Java
>>> 122: * platform uses the UTF-16 representation in {@code char} arrays and
>>> 123: * in the {
On Thu, 9 Sep 2021 06:50:21 GMT, Andrey Turbanov
wrote:
> StringBuffer is a legacy synchronized class. There are more modern
> alternatives which perform better:
> 1. Plain String concatenation should be preferred
> 2. StringBuilder is a direct replacement to StringBuffer which generally have
On Thu, 9 Sep 2021 20:12:47 GMT, Andrey Turbanov
wrote:
> Redundant castings make code harder to read.
> Found them by IntelliJ IDEA.
> I tried to select only casts which are definitely safe to remove. Also didn't
> touch primitive types casts.
Calendar-related changes look good to me.
--
On Thu, 8 Jul 2021 21:23:00 GMT, Naoto Sato wrote:
> This is an implementation for the `JEP 400: UTF-8 by Default`. The gist of
> the changes is `Charset.defaultCharset()` returning `UTF-8` and
> `file.encoding` system property being added in the spec, but another notable
> modifi
> CSR: https://bugs.openjdk.java.net/browse/JDK-8260266
Naoto Sato has updated the pull request incrementally with one additional
commit since the last revision:
Fixed a failing test
-
Changes:
- all: https://git.openjdk.java.net/jdk/pull/4733/files
- new: https://git.openj
> CSR: https://bugs.openjdk.java.net/browse/JDK-8260266
Naoto Sato has updated the pull request with a new target base due to a merge
or a rebase. The incremental webrev excludes the unrelated changes brought in
by the merge/rebase. The pull request contains 15 additional commits since the
la
On Mon, 23 Aug 2021 21:01:48 GMT, Andrey Turbanov
wrote:
> Collections.sort is just a wrapper, so it is better to use an instance method
> directly.
java.time changes look good.
-
Marked as reviewed by naoto (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/5229
On Sun, 22 Aug 2021 23:02:06 GMT, Sergey Bylokhov wrote:
>> This is the continuation of JDK-8233884, JDK-8271456, and JDK-8272120.
>>
>> In many places standard charsets are looked up via their names, for example:
>> absolutePath.getBytes("UTF-8");
>>
>> This could be done more efficiently(up t
> CSR: https://bugs.openjdk.java.net/browse/JDK-8260266
Naoto Sato has updated the pull request with a new target base due to a merge
or a rebase. The incremental webrev excludes the unrelated changes brought in
by the merge/rebase. The pull request contains 12 additional commits since the
On Tue, 10 Aug 2021 05:08:54 GMT, Sergey Bylokhov wrote:
> This is the continuation of JDK-8233884 and JDK-8271456. This change affects
> fewer cases so I fix all "java." modules at once.
>
> In many places standard charsets are looked up via their names, for example:
> absolutePath.getBytes("U
On Tue, 10 Aug 2021 05:08:54 GMT, Sergey Bylokhov wrote:
> This is the continuation of JDK-8233884 and JDK-8271456. This change affects
> fewer cases so I fix all "java." modules at once.
>
> In many places standard charsets are looked up via their names, for example:
> absolutePath.getBytes("U
> CSR: https://bugs.openjdk.java.net/browse/JDK-8260266
Naoto Sato has updated the pull request incrementally with one additional
commit since the last revision:
Removed leftover `console` references in `PrintStream`
-
Changes:
- all: https://git.openjdk.jav
On Wed, 14 Jul 2021 20:52:54 GMT, Naoto Sato wrote:
>> src/java.base/share/classes/java/io/Console.java line 587:
>>
>>> 585: try {
>>> 586: cs = Charset.forName(csname);
>>> 587: } catch (Exception ign
> CSR: https://bugs.openjdk.java.net/browse/JDK-8260266
Naoto Sato has updated the pull request incrementally with one additional
commit since the last revision:
Reflects comments
-
Changes:
- all: https://git.openjdk.java.net/jdk/pull/4733/files
- new: https://git.openj
On Wed, 14 Jul 2021 21:03:53 GMT, Roger Riggs wrote:
>> That was intentional. Only those two are supported, others continue to work
>> as before (but not supported).
>
> Still it leaves an uncomfortable feeling, perhaps remedied by an "other
> values have unspecified behavior"
> or the "other v
On Wed, 14 Jul 2021 16:20:28 GMT, Jesse Glick
wrote:
>> src/java.base/share/classes/java/io/PrintStream.java line 49:
>>
>>> 47: * All characters printed by a {@code PrintStream} are converted
>>> into
>>> 48: * bytes using the given encoding or charset, or the default
>>> 49: * console ch
On Wed, 14 Jul 2021 14:55:39 GMT, Roger Riggs wrote:
>> Naoto Sato has updated the pull request with a new target base due to a
>> merge or a rebase. The incremental webrev excludes the unrelated changes
>> brought in by the merge/rebase. The pull request contains three addi
> CSR: https://bugs.openjdk.java.net/browse/JDK-8260266
Naoto Sato has updated the pull request with a new target base due to a merge
or a rebase. The incremental webrev excludes the unrelated changes brought in
by the merge/rebase. The pull request contains three additional commits since
On Wed, 14 Jul 2021 12:39:46 GMT, Giacomo Baso
wrote:
> > Consider an application that creates a java.io.FileWriter with its
> > one-argument constructor and then uses it to write some text to a file. The
> > resulting file will contain a sequence of bytes encoded using the default
> > charse
This is an implementation for the `JEP 400: UTF-8 by Default`. The gist of the
changes is `Charset.defaultCharset()` returning `UTF-8` and `file.encoding`
system property being added in the spec, but another notable modification is in
`java.io.PrintStream` where it continues to use the `Console`
On Fri, 25 Jun 2021 20:04:37 GMT, Weijun Wang wrote:
> More refactoring to limit the scope of `@SuppressWarnings` annotations.
>
> Sometimes I introduce new methods. Please feel free to suggest method names
> you like to use.
LGTM.
-
Marked as reviewed by naoto (Reviewer).
PR: h
On Mon, 17 May 2021 18:23:41 GMT, Weijun Wang wrote:
> Please review this implementation of [JEP
> 411](https://openjdk.java.net/jeps/411).
>
> The code change is divided into 3 commits. Please review them one by one.
>
> 1.
> https://github.com/openjdk/jdk/commit/576161d15423f58281e384174d28
On Tue, 27 Apr 2021 21:34:02 GMT, Ian Graves wrote:
> 8266155: Convert java.base to use Stream.toList()
Marked as reviewed by naoto (Reviewer).
-
PR: https://git.openjdk.java.net/jdk/pull/3734
On Fri, 26 Feb 2021 10:59:01 GMT, Daniel Fuchs wrote:
>> Hi,
>>
>> Please find here a change that fixes "no comment" warnings generated by
>> `javadoc -Xdoclint` for `java.base/java.net`
>
> Daniel Fuchs has updated the pull request incrementally with one additional
> commit since the last re
On Thu, 25 Feb 2021 11:05:07 GMT, Daniel Fuchs wrote:
>> Hi,
>>
>> Please find here a change that fixes "no comment" warnings generated by
>> `javadoc -Xdoclint` for `java.base/java.net`
>
> Daniel Fuchs has updated the pull request incrementally with one additional
> commit since the last re
On Fri, 8 Jan 2021 20:34:10 GMT, Roger Riggs wrote:
> Cleanup of tests test/jdk/java/net and test/jdk/sun/net that format
> hexadecimal strings to use java.util.HexFormat methods.
> Also in tests test/jdk/java/util/Locale/SoftKeys.
+1
-
Marked as reviewed by naoto (Reviewer).
PR:
Hi Mikael,
I took a look at i18n related files. It looks good overall.
One nit in java/nio/charset/Charset/DefaultCharsetTest.java: If the test
is only applicable to linux (@requires os.family == "linux" in jtreg tag
after the change), the condition isLinux() is no longer needed at line 73.
16:39, Naoto Sato wrote:
Hi Pavel,
Locale.setDefault() sets the JVM wide default locale, so I'd suggest add
try/finally block to resume the locale to the previous one.
I did suggest something like this to Pavel off-list, but then
noticed that the test runs in othervm mode, so won't af
Hi Pavel,
Locale.setDefault() sets the JVM wide default locale, so I'd suggest add
try/finally block to resume the locale to the previous one.
Naoto
On 6/13/16 3:29 AM, Pavel Rappo wrote:
Hi,
Could you please review the following change for JDK-8159039?
http://cr.openjdk.java.net/~prappo/8
Pinging. Can anyone please review this simple fix?
Naoto
On 3/18/16 10:53 AM, Naoto Sato wrote:
Hello,
Please review the fix to the following bug:
https://bugs.openjdk.java.net/browse/JDK-8060097
The proposed fix is located at:
http://cr.openjdk.java.net/~naoto/8060097/webrev.00/
The test
Hello,
Please review the fix to the following bug:
https://bugs.openjdk.java.net/browse/JDK-8060097
The proposed fix is located at:
http://cr.openjdk.java.net/~naoto/8060097/webrev.00/
The test was failing due to the infamous Turkish 'i' upper/lowercasing.
Providing the casing operations wit
Hello,
I'd suggest using Locale.ROOT instead of Locale.ENGLISH for this purpose.
Naoto
On 1/21/16 10:57 PM, Claes Redestad wrote:
Hi,
using String.toLowerCase() in URL allows some strings to lower-case to a
string that'd be invalid, e.g. "FILE".toLowerCase() -> "f\u0131le" in
turkish locales.
Changeset: 5b5ee2288306
Author:naoto
Date: 2013-12-02 11:29 -0800
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/5b5ee2288306
8028368: There is no description whether or not java.util.ResourceBundle is
thread-safe
Reviewed-by: okutsu
! src/share/classes/java/util/ListResourceBund
Changeset: 2370d285d08b
Author:naoto
Date: 2013-11-27 10:01 -0800
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/2370d285d08b
8028771: regression test java/util/Locale/LocaleProviders.sh failed
Reviewed-by: alanb
! test/java/util/Locale/LocaleProviders.java
! test/java/util/Locale
Changeset: fb7abd509bd2
Author:naoto
Date: 2013-11-07 10:03 -0800
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/fb7abd509bd2
8027930: ResourceBundle test failures in fr locale
Reviewed-by: smarks
! test/java/util/ResourceBundle/ResourceBundleTest.java
! test/java/util/ResourceBun
Changeset: b4c259743371
Author:naoto
Date: 2013-09-30 16:15 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/b4c259743371
8016110: Japanese char (MS932) 0x5C cannot be used as an argument when quoted
Reviewed-by: ksrini, akhil
! src/windows/bin/cmdtoargs.c
+ test/tools/launche
Changeset: 7bfe3da4fad6
Author:naoto
Date: 2013-09-11 05:38 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/7bfe3da4fad6
8024332: sun/util/resources/en split between rt.jar and localedata.jar
Reviewed-by: alanb, erikj
! make/java/java/genlocales.gmk
! make/java/java/localegen
Changeset: 4c711ef41bfa
Author:naoto
Date: 2013-09-05 10:14 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/4c711ef41bfa
8023943: Method description fix for String.toLower/UpperCase() methods
Reviewed-by: okutsu
! src/share/classes/java/lang/String.java
Changeset: 883cc296ec89
Author:bchristi
Date: 2013-08-02 15:30 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/883cc296ec89
8011194: Apps launched via double-clicked .jars have file.encoding value of
US-ASCII on Mac OS X
Summary: On Mac, default to UTF-8 if no environmental h
Changeset: e34e3ddb3cd8
Author:naoto
Date: 2013-06-27 14:40 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/e34e3ddb3cd8
6609431: (rb) ResourceBundle.getString() returns incorrect value
Reviewed-by: okutsu, sherman
! src/share/classes/java/util/Properties.java
+ test/java/uti
Changeset: 336e5a862013
Author:naoto
Date: 2013-06-26 11:21 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/336e5a862013
8017322: java/util/Currency/PropertiesTest.sh should run exclusively
Reviewed-by: alanb
! test/TEST.ROOT
Changeset: bb2e67628dc0
Author:naoto
Date: 2013-06-24 16:21 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/bb2e67628dc0
8017468: typo in javadoc: " ResourceBunlde "
Reviewed-by: okutsu
! src/share/classes/java/util/spi/LocaleServiceProvider.java
Changeset: 8b84d557570c
Author:naoto
Date: 2013-06-21 13:42 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/8b84d557570c
6863624: java/util/Currency/PropertiesTest.sh writable check is incorrect
Reviewed-by: alanb
! test/java/util/Currency/PropertiesTest.sh
! test/java/util/L
Changeset: cadb0ef6e931
Author:naoto
Date: 2013-06-11 11:18 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/cadb0ef6e931
8015960: java/util/Locale/LocaleProviders.java failing again on Windows
Reviewed-by: alanb
! test/java/util/Locale/LocaleProviders.java
Changeset: 379e1bcae693
Author:naoto
Date: 2013-06-04 10:33 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/379e1bcae693
8013903: Japanese calendar field names are not displayed with
-Djava.locale.providers=HOST on Windows
Reviewed-by: okutsu
! src/share/classes/java/util/sp
Changeset: b99d56d1aa3f
Author:naoto
Date: 2013-05-28 14:02 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/b99d56d1aa3f
6251788: (rb) PropertyResourceBundle doesn't document exceptions
Reviewed-by: okutsu
! src/share/classes/java/util/PropertyResourceBundle.java
Changeset: 50fde3eeb48c
Author:naoto
Date: 2013-05-22 16:43 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/50fde3eeb48c
7056126: DateFormatSymbols documentation has incorrect description about
DateFormat
7083668: Sample code in ListResourceBundle is still not correct
Reviewe
Changeset: e30396e22c6f
Author:naoto
Date: 2013-05-07 11:31 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/e30396e22c6f
8013086: NPE thrown by SimpleDateFormat with TimeZoneNameProvider supplied
Reviewed-by: okutsu
! src/share/classes/sun/util/locale/provider/TimeZoneNameUti
Changeset: 62fb9e2b5da1
Author:naoto
Date: 2013-04-22 13:37 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/62fb9e2b5da1
8010666: Implement Currency/LocaleNameProvider in Windows Host
LocaleProviderAdapter
Reviewed-by: okutsu
! src/macosx/classes/sun/util/locale/provider/Hos
Changeset: 9a6ef3391f32
Author:naoto
Date: 2013-04-03 10:32 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/9a6ef3391f32
7091601: Arabic Locale: can not set type of digit in application level
Reviewed-by: okutsu
! src/macosx/native/sun/util/locale/provider/HostLocaleProviderA
Changeset: d79503c4c56f
Author:naoto
Date: 2013-03-14 11:29 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/d79503c4c56f
8008576: Calendar mismatch using Host LocaleProviderAdapter
Reviewed-by: okutsu
! make/java/java/FILES_java.gmk
! src/macosx/classes/sun/util/locale/provid
Changeset: 1a2e59d19d3e
Author:naoto
Date: 2013-03-04 20:46 -0800
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/1a2e59d19d3e
8004240: Return value from getAdapterPrefence() can be modified
Reviewed-by: okutsu
! src/share/classes/sun/util/locale/provider/LocaleProviderAdapter.java
Changeset: af396ec087f4
Author:naoto
Date: 2013-02-19 10:34 -0800
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/af396ec087f4
7092447: Clarify the default locale used in each locale sensitive operation
Reviewed-by: okutsu
! src/share/classes/java/text/DateFormat.java
! src/share/c
Changeset: 79d7595abe95
Author:naoto
Date: 2013-02-08 09:35 -0800
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/79d7595abe95
8007038: ArrayIndexOutOfBoundsException on calling localizedDateTime().print()
with JapaneseChrono
Reviewed-by: okutsu
! src/share/classes/sun/util/locale
Changeset: 1d7a6adf499f
Author:naoto
Date: 2013-01-14 11:09 -0800
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/1d7a6adf499f
7162007: Clean up i18n related caches
Reviewed-by: okutsu, ohair
! make/java/java/FILES_java.gmk
! src/share/classes/java/text/DateFormatSymbols.java
! src
Changeset: c622df692bfb
Author:bchristi
Date: 2013-01-10 10:21 -0800
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/c622df692bfb
8005962: TEST_BUG: java/util/Properties/MacJNUEncoding can fail in certain
environments
Summary: Test script now sets LC_ALL, other small changes, reloc
Changeset: dbc692ea3f0a
Author:bchristi
Date: 2013-01-07 13:19 -0800
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/dbc692ea3f0a
8003228: (props) sun.jnu.encoding should be set to UTF-8 [macosx]
Summary: Hard-code sun.jnu.encoding to UTF-8 on Mac
Reviewed-by: naoto
! src/share/nat
Changeset: 64a42798ea5e
Author:naoto
Date: 2012-11-15 20:17 -0800
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/64a42798ea5e
7199750: Loading sequence of service provider is changed
Reviewed-by: okutsu
! src/share/classes/sun/util/locale/provider/SPILocaleProviderAdapter.java
! t
Changeset: cc325832469c
Author:naoto
Date: 2012-11-07 15:08 -0800
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/cc325832469c
8001205: Calendar.getDisplayName(...): Returns null when provider is SPI but
there is no SPI implementation
8001562: Collator.getAvailableLocales() doesn't
Changeset: 6420fcd61c10
Author:naoto
Date: 2012-11-01 13:28 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/6420fcd61c10
8001440: CLDR adapter: Invalid number extension in language tag causes
exception in NumberFormat.format()
Reviewed-by: okutsu
! src/share/classes/java/tex
Changeset: 7fa45c455034
Author:naoto
Date: 2012-10-29 10:42 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/7fa45c455034
8000997: Multiple locale sensitive services cannot be loaded
Reviewed-by: okutsu
! src/share/classes/sun/util/locale/provider/LocaleServiceProviderPool.jav
Changeset: 60994591be6b
Author:naoto
Date: 2012-10-17 13:22 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/60994591be6b
8001046: java/util/PluggableLocale/LocaleNameProviderTest.sh failing
Reviewed-by: okutsu
! test/java/util/PluggableLocale/barprovider.jar
Changeset: 32452042b781
Author:naoto
Date: 2012-10-16 10:59 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/32452042b781
8000245: SimpleDateFormat.format(date, StringBuffer, FieldPosition) doesn't
work as expected with custom extensions
8000273: java.util.Locale.getDisplayVar
Changeset: c725ce4bbf12
Author:naoto
Date: 2012-10-09 09:59 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/c725ce4bbf12
7200341: DateFormatSymbols.hashCode() throws ArrayIndexOutOfBoundsException in
some circumstances
Reviewed-by: okutsu
! src/share/classes/java/text/DateFo
Changeset: 88a726a5b2dc
Author:naoto
Date: 2012-10-05 09:57 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/88a726a5b2dc
7198834: HOST Adapter: one extra empty space in the end of the pattern string
Reviewed-by: okutsu
! src/share/classes/sun/util/locale/provider/JRELocalePro
Changeset: cd4f181eb666
Author:naoto
Date: 2012-10-04 21:03 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/cd4f181eb666
7200119: Collator.getAvailableLocales() doesn't return Locale.US
Reviewed-by: okutsu
! src/share/classes/sun/util/locale/provider/JRELocaleProviderAdapter.
Changeset: c6a0b13e6efa
Author:naoto
Date: 2012-10-04 10:04 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/c6a0b13e6efa
7196799: CLDR adapter can not be invoked when region code is specified in Locale
7197573: java/util/Locale/LocaleProviders.sh failed.
Reviewed-by: okutsu
!
Changeset: 0136fca60652
Author:naoto
Date: 2012-09-18 10:34 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/0136fca60652
7198984: Add java/text/Bidi/Bug6665028.java to ProblemList.txt
Reviewed-by: alanb
Contributed-by: yiming.w...@oracle.com
! test/ProblemList.txt
Changeset: faf4528aea4e
Author:naoto
Date: 2012-08-24 10:13 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/faf4528aea4e
7193601: Build breakage with the fix to 6336885 (build-infra build)
Reviewed-by: mduigou
! makefiles/CompileJavaClasses.gmk
Changeset: b15db7ded589
Author:naoto
Date: 2012-03-27 10:10 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/b15db7ded589
7156459: Remove unnecessary get() from Currency.getInstance()
Reviewed-by: chegar, dholmes, mduigou
! src/share/classes/java/util/Currency.java
Changeset: 4a5817f9e249
Author:naoto
Date: 2012-03-21 10:10 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/4a5817f9e249
7145454: JVM wide monitor lock in Currency.getInstance(String)
Reviewed-by: okutsu
! src/share/classes/java/util/Currency.java
Changeset: c647ebb3c4f7
Author:naoto
Date: 2011-12-13 15:41 -0800
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/c647ebb3c4f7
4808233: "Locale" not thread-safe
Reviewed-by: okutsu
! src/share/classes/java/util/Locale.java
Changeset: 85363edbc92f
Author:naoto
Date: 2011-12-05 17:08 -0800
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/85363edbc92f
7117469: Warning cleanup for j.u.Currency and j.u.Locale related classes
Reviewed-by: okutsu, smarks
! src/share/classes/java/util/Currency.java
! src/shar
Changeset: 42532a097816
Author:naoto
Date: 2011-12-02 16:04 -0800
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/42532a097816
7117465: Warning cleanup for IMF classes
Reviewed-by: okutsu
! src/share/classes/java/awt/im/InputMethodHighlight.java
! src/share/classes/sun/awt/im/Compo
Changeset: 829c3a8d23fa
Author:naoto
Date: 2011-10-12 12:12 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/829c3a8d23fa
7027061: Testcase failure: java/util/Locale/Bug6989440.java -
java.util.ConcurrentModificationException
Reviewed-by: dholmes, chegar
! src/share/classes/s
Changeset: b8a1d30d6c65
Author:naoto
Date: 2011-10-06 17:40 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/b8a1d30d6c65
7098394: JDK8 TL repo build fails in src/solaris/native/java/util/TimeZone_md.c
Reviewed-by: chegar
! src/solaris/native/java/util/TimeZone_md.c
Changeset: c3a8d5feee94
Author:naoto
Date: 2011-08-17 11:09 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/c3a8d5feee94
7073906: Locale.getDefault() returns wrong Locale for Java SE 7
Reviewed-by: okutsu
! src/windows/native/java/lang/java_props_md.c
Changeset: f59c83f515e6
Changeset: 42fe05e54e69
Author:naoto
Date: 2011-07-12 10:28 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/42fe05e54e69
7022407: Spinning CPU in LocaleObjectCache.get()
Reviewed-by: okutsu
! src/share/classes/sun/util/locale/LocaleObjectCache.java
Changeset: 05939afe3fc2
Author:naoto
Date: 2011-05-09 13:30 -0700
URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/05939afe3fc2
7041950: Fix copyright
Reviewed-by: okutsu
! src/share/classes/sun/text/resources/BreakIteratorRules_th.java
Changeset: ea45b4ed1758
Author:naoto
Date: 2011-04-06 10:53 -0700
URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/ea45b4ed1758
7031546: test/java/util/ResourceBundle/Bug4168625Test.java fails on solaris10u9
sparc.
Reviewed-by: okutsu
! test/java/util/ResourceBundle/Bug4168625Test.
1 - 100 of 102 matches
Mail list logo
