Masayoshi, Michael,
Thank you for the review.

Sean,
Thank you for the approval.

With Best Regards,
Aleksej

On 07/12/2014 04:37 PM, Masayoshi Okutsu wrote:
Looks good to me.

Thanks,
Masayoshi

On 6/4/2014 3:53 AM, Seán Coffey wrote:
Approved for jdk7u-dev but subject to reviewer approval also. I'm cc'ing the original JDK 8 reviewers. Hopefully one of them can approve.

regards,
Sean.

On 03/06/2014 18:10, Aleksej Efimov wrote:
Hello,

Can I have an approval for backport of the time zone names changes [1] to JDK7u-dev. The backport slightly differs from the original JDK9 changes: 1. There is no generic names in JDK7 and because of that only the STD/DST naming changes was backported. 2. The test that was introduced by original bug was removed from JDK9 [2] - thats why it wasn't backported.
The webrev is located here [3].
The following set of tests was executed on JDK7 build with proposed fix: test/sun/util/resources/TimeZone test/sun/util/calendar test/closed/java/util/TimeZone test/java/util/TimeZone test/java/util/Calendar test/java/util/Formatter test/sun/util/resources/TimeZone

Thank you,
Aleksej

[1] JDK8 changeset: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/46c727d6ecc2 [2] JDK8 changeset with removed test: https://bugs.openjdk.java.net/browse/JDK-8033785
[3] Webrev: http://cr.openjdk.java.net/~aefimov/8025051/7/webrev.00/
[4] JBS bug: https://bugs.openjdk.java.net/browse/JDK-8025051



Reply via email to