Re: Patch to expand tz checking scope in TimeZone_md.c

2011-11-03 Thread David Holmes
On 4/11/2011 2:53 PM, David Holmes wrote: On 4/11/2011 2:13 PM, Masayoshi Okutsu wrote: Probably the difference isn't documented. I tried Solaris 10 and Ubuntu 10.03. The difference still exists. Solaris 10: $ unset TZ $ date Fri Nov 4 13:04:45 JST 2011 $ TZ="" date Fri Nov 4 13:04:53 JST 2011

Re: Patch to expand tz checking scope in TimeZone_md.c

2011-11-03 Thread David Holmes
On 4/11/2011 2:13 PM, Masayoshi Okutsu wrote: Probably the difference isn't documented. I tried Solaris 10 and Ubuntu 10.03. The difference still exists. Solaris 10: $ unset TZ $ date Fri Nov 4 13:04:45 JST 2011 $ TZ="" date Fri Nov 4 13:04:53 JST 2011 Ubuntu 10.04: $ unset TZ $ date Fri Nov 4

Re: Patch to expand tz checking scope in TimeZone_md.c

2011-11-03 Thread Masayoshi Okutsu
Probably the difference isn't documented. I tried Solaris 10 and Ubuntu 10.03. The difference still exists. Solaris 10: $ unset TZ $ date Fri Nov 4 13:04:45 JST 2011 $ TZ="" date Fri Nov 4 13:04:53 JST 2011 Ubuntu 10.04: $ unset TZ $ date Fri Nov 4 13:05:50 JST 2011 $ TZ="" date Fri Nov 4 0

Re: Code Review Request for 4533691 (add Collections.EMPTY_SORTED_SET)

2011-11-03 Thread David Holmes
Mike, I see that you have pushed a version of this change and that I was listed as a reviewer. However I never saw an updated webrev and there was no response to my query below. In fact I never saw any response to any of the reviewers comments on this. David On 31/10/2011 10:52 AM, David Ho

hg: jdk8/tl/jdk: 4533691: Add Collections.emptySortedSet()

2011-11-03 Thread mike . duigou
Changeset: 2f2f56ac8b82 Author:mduigou Date: 2011-11-03 13:26 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/2f2f56ac8b82 4533691: Add Collections.emptySortedSet() Reviewed-by: mduigou, alanb, dholmes Contributed-by: darryl.mo...@oracle.com ! src/share/classes/java/util/Coll

Re: Patch to expand tz checking scope in TimeZone_md.c

2011-11-03 Thread Jonathan Lu
Hi Masayoshi, I did find some references about date-time related functions / TZ variables on Linux but got only a few about Solaris, so could not see any differences between those two platforms about the changes described in my patch. Have you got any links or references about these differenc