pgsql: Check return code from strxfrm on Windows since it has a

Lists: pgsql-committers
From: mha(at)postgresql(dot)org (Magnus Hagander)
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Check return code from strxfrm on Windows since it has a
Date: 2007-05-05 17:05:49
Message-ID: 20070505170549.393649FB6F4@postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-committers

Log Message:
-----------
Check return code from strxfrm on Windows since it has a
non-standard way of indicating errors, so we don't try to
allocate INT_MAX bytes to store a result in.

Modified Files:
--------------
pgsql/src/backend/utils/adt:
selfuncs.c (r1.233 -> r1.234)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/selfuncs.c.diff?r1=1.233&r2=1.234)


From: Bruce Momjian <bruce(at)momjian(dot)us>
To: Magnus Hagander <mha(at)postgresql(dot)org>
Cc: pgsql-committers(at)postgresql(dot)org
Subject: Re: pgsql: Check return code from strxfrm on Windows since it has a
Date: 2007-05-06 02:31:55
Message-ID: 200705060231.l462VtA24192@momjian.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-committers


FYI, you forgot the author's in the commit message, ITAGAKI Takahiro.

---------------------------------------------------------------------------

Magnus Hagander wrote:
> Log Message:
> -----------
> Check return code from strxfrm on Windows since it has a
> non-standard way of indicating errors, so we don't try to
> allocate INT_MAX bytes to store a result in.
>
> Modified Files:
> --------------
> pgsql/src/backend/utils/adt:
> selfuncs.c (r1.233 -> r1.234)
> (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/selfuncs.c.diff?r1=1.233&r2=1.234)
>
> ---------------------------(end of broadcast)---------------------------
> TIP 9: In versions below 8.0, the planner will ignore your desire to
> choose an index scan if your joining column's datatypes do not
> match

--
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. +


From: Magnus Hagander <magnus(at)hagander(dot)net>
To: Bruce Momjian <bruce(at)momjian(dot)us>
Cc: pgsql-committers(at)postgresql(dot)org
Subject: Re: pgsql: Check return code from strxfrm on Windows since it has a
Date: 2007-05-06 07:17:20
Message-ID: 463D8100.2060100@hagander.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-committers

This is actually a different patch. I'm still waiting for confirmation
on if it's enough or if we need both this patch and the original, or if
this one is enough to solve his problem.

Still, I did forget the report-and-research attribution. My apologies.

//Magnus

Bruce Momjian wrote:
> FYI, you forgot the author's in the commit message, ITAGAKI Takahiro.
>
> ---------------------------------------------------------------------------
>
> Magnus Hagander wrote:
>> Log Message:
>> -----------
>> Check return code from strxfrm on Windows since it has a
>> non-standard way of indicating errors, so we don't try to
>> allocate INT_MAX bytes to store a result in.
>>
>> Modified Files:
>> --------------
>> pgsql/src/backend/utils/adt:
>> selfuncs.c (r1.233 -> r1.234)
>> (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/selfuncs.c.diff?r1=1.233&r2=1.234)
>>
>> ---------------------------(end of broadcast)---------------------------
>> TIP 9: In versions below 8.0, the planner will ignore your desire to
>> choose an index scan if your joining column's datatypes do not
>> match
>