Re: FW: RFR(XS): 8133830: [solaris] Fix for potential memory leak in TimeZone_md.c, function findJavaTZ_md()

2015-08-24 Thread Masayoshi Okutsu
Looks good to me too. Thank you for fixing this bug. Aleksej, it's great if you can sponsor this fix. Thanks, Masayoshi On 8/25/2015 1:27 AM, Aleksej Efimov wrote: Hi Christoph, I can sponsor your change. Fix looks good to me (but I'm not a reviewer). With Best Regards, Aleksej On 08/24/2015

Re: [9] RFR: 8134250: Create unit tests for CLDR unique features

2015-08-24 Thread Naoto Sato
Thanks. I found that the error message in the second RuntimeException was somewhat incorrect, so will correct it before pushing the changeset. Naoto On 8/24/15 4:31 PM, Masayoshi Okutsu wrote: Looks good to me. Thanks for adding this unit test. Masayoshi On 8/25/2015 7:18 AM, Naoto Sato wrot

Re: [9] RFR: 8134250: Create unit tests for CLDR unique features

2015-08-24 Thread Masayoshi Okutsu
Looks good to me. Thanks for adding this unit test. Masayoshi On 8/25/2015 7:18 AM, Naoto Sato wrote: Hello, Please review the changes for the following issue: https://bugs.openjdk.java.net/browse/JDK-8134250 It is just adding more unit testcases for the JEP 252. The proposed changeset is l

[9] RFR: 8134250: Create unit tests for CLDR unique features

2015-08-24 Thread Naoto Sato
Hello, Please review the changes for the following issue: https://bugs.openjdk.java.net/browse/JDK-8134250 It is just adding more unit testcases for the JEP 252. The proposed changeset is located at: http://cr.openjdk.java.net/~naoto/8134250/webrev.00/ Naoto

Re: FW: RFR(XS): 8133830: [solaris] Fix for potential memory leak in TimeZone_md.c, function findJavaTZ_md()

2015-08-24 Thread Aleksej Efimov
Hi Christoph, I can sponsor your change. Fix looks good to me (but I'm not a reviewer). With Best Regards, Aleksej On 08/24/2015 12:24 PM, Langer, Christoph wrote: Hi, I'm also posting this RFR here. Is this the right place for my change? Is anyone willing to sponsor this? Thanks a lot in ad

FW: RFR(XS): 8133830: [solaris] Fix for potential memory leak in TimeZone_md.c, function findJavaTZ_md()

2015-08-24 Thread Langer, Christoph
Hi, I'm also posting this RFR here. Is this the right place for my change? Is anyone willing to sponsor this? Thanks a lot in advance Christoph From: Langer, Christoph Sent: Donnerstag, 20. August 2015 12:22 To: 'core-libs-...@openjdk.java.net' Subject: RFR(XS): 8133830: [solaris] Fix for pote