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
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:
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
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
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
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
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
>
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:
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
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
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
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:
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`
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
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
>
> 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
> 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:
>
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:
>
> 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
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
> >
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
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:
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.
>
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
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
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).
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.
, 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 affect
subsequent
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
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
!
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
!
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
!
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
+
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
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
+
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
!
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
!
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
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
!
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
!
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
!
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
!
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
!
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
!
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,
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
!
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
!
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()
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
!
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:
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
!
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
!
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: 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: 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
!
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
!
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:
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
!
Changeset: 8216ad5fabb8
Author:naoto
Date: 2011-03-11 11:31 -0800
URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/8216ad5fabb8
7023613: (lc) Incorrect string returned by Locale.toString() with non-empty
script field
Reviewed-by: srl
! src/share/classes/java/util/Locale.java
!
Changeset: ecff24121064
Author:naoto
Date: 2010-08-25 15:31 -0700
URL: http://hg.openjdk.java.net/jdk7/tl/langtools/rev/ecff24121064
6875847: Java Locale Enhancement
Summary: Fix for javac to allow sun.util.locale package accessible.
Reviewed-by: jjg
!
85 matches
Mail list logo