From: | "Hiroshi Saito" <z-saito(at)guitar(dot)ocn(dot)ne(dot)jp> |
---|---|
To: | "Jaime Casanova" <jcasanov(at)systemguards(dot)com(dot)ec> |
Cc: | "ITAGAKI Takahiro" <itagaki(dot)takahiro(at)oss(dot)ntt(dot)co(dot)jp>, "Alvaro Herrera" <alvherre(at)commandprompt(dot)com>, "Magnus Hagander" <magnus(at)hagander(dot)net>, "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us>, "pgsql-hackers" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [PATCHES] Solve a problem of LC_TIME of windows. |
Date: | 2008-11-24 07:02:08 |
Message-ID: | 6C4CA7280B9946D786A7A543D4D3AB47@HIRO57887DE653 |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers pgsql-patches |
Hi Jaime-san.
Thank you for a review.
I think this purpose to return the value which should originally obtain strftime
by only replacing here. Then, I think that it is a superfluous reaction.
However, some consideration may be necessities.
Regards,
Hiroshi Saito
----- Original Message -----
From: "Jaime Casanova" <jcasanov(at)systemguards(dot)com(dot)ec>
On Sun, Nov 16, 2008 at 8:36 AM, Hiroshi Saito <z-saito(at)guitar(dot)ocn(dot)ne(dot)jp> wrote:
> Hi.
>
> Then, It continues and a review is desired. Thanks!
>
In http://msdn.microsoft.com/en-us/library/fe06s4ak(VS.71).aspx says:
"""
Return Value
strftime returns the number of characters placed in strDest and
wcsftime returns the corresponding number of wide characters.
If the total number of characters, including the terminating null, is
more than maxsize, both strftime and wcsftime return 0 and the
contents of strDest is indeterminate.
"""
If i'm reading it right, the patch should contain something like:
if (len > dstlen)
{
return 0;
}
--
Atentamente,
Jaime Casanova
Soporte y capacitación de PostgreSQL
Asesoría y desarrollo de sistemas
Guayaquil - Ecuador
Cel. +59387171157
From | Date | Subject | |
---|---|---|---|
Next Message | Hiroshi Saito | 2008-11-24 07:17:40 | Re: pgsql: Explicitly bind gettext() to the UTF8 locale when in use. |
Previous Message | Jaime Casanova | 2008-11-24 04:13:19 | Re: [PATCHES] Solve a problem of LC_TIME of windows. |
From | Date | Subject | |
---|---|---|---|
Next Message | Magnus Hagander | 2008-11-24 13:02:05 | Re: [PATCHES] Solve a problem of LC_TIME of windows. |
Previous Message | Jaime Casanova | 2008-11-24 04:13:19 | Re: [PATCHES] Solve a problem of LC_TIME of windows. |