From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Brendan Jurd <direvus(at)gmail(dot)com>, PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: [HACKERS] Error in from_char() for field 'D'? |
Date: | 2007-02-14 05:11:27 |
Message-ID: | 200702140511.l1E5BR228980@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers pgsql-patches |
Tom Lane wrote:
> "Brendan Jurd" <direvus(at)gmail(dot)com> writes:
> > However, in from_char(), the reverse is not true. Looking at the code
> > snippet above, the digit is scanned straight into tmfc->d unaltered
> > (this value is later copied directly to tm->tm_wday circa line 3394).
> > Unless I'm missing something, when converting to text, 'D' yields 1-7,
> > but when converting back from text, 'D' expects 0-6.
>
> Although this does look like a bug, I'm not sure it matters, because
> AFAICS there is no code path that will look at the value of tm_wday
> while constructing a timestamp value from a struct tm. I'm inclined
> not to risk messing with it just before RC1 unless a visible fault
> can be demonstrated.
Fixed in 8.3, patch attached.
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://www.enterprisedb.com
+ If your life is a hard drive, Christ can be your backup. +
Attachment | Content-Type | Size |
---|---|---|
/rtmp/diff | text/x-diff | 568 bytes |
From | Date | Subject | |
---|---|---|---|
Next Message | Jeroen T. Vermeulen | 2007-02-14 06:28:52 | Cosmetic note: hit rates in logs |
Previous Message | Bruce Momjian | 2007-02-14 05:01:27 | Re: [PERFORM] Direct I/O issues |
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Eisentraut | 2007-02-14 07:11:12 | Re: [GENERAL] ISO week dates |
Previous Message | Bruce Momjian | 2007-02-14 05:01:27 | Re: [PERFORM] Direct I/O issues |