Re: [HACKERS] Patch for VS.Net 2005's strxfrm() bug

From: "Hiroshi Saito" <z-saito(at)guitar(dot)ocn(dot)ne(dot)jp>
To: "Florian G(dot) Pflug" <fgp(at)phlo(dot)org>, "Bruce Momjian" <bruce(at)momjian(dot)us>
Cc: "Andrew Dunstan" <andrew(at)dunslane(dot)net>, <pgsql-hackers(at)postgresql(dot)org>, "William ZHANG" <uniware(at)zedware(dot)org>, "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us>, <pgsql-patches(at)postgresql(dot)org>
Subject: Re: [HACKERS] Patch for VS.Net 2005's strxfrm() bug
Date: 2006-07-27 00:56:01
Message-ID: 041801c6b117$6fc54040$e8100dde@IBMC4B5932F74B
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

From: "Florian G. Pflug"

Ahhhhhhh, It is right.!
I was retracing my memory for what situations the contents were.
I was in distraction.....It seems that it is satisfactory at the reason for ==.

Sorry and Thanks.!!

Regards,
Hiroshi Saito

> Bruce Momjian wrote:
>>>> Why is this better than:
>>>>
>>>> #if _MSC_VER == 1400
>>>>
>>>>
>>>> Surely this will not be true if _MSC_VER is undefined?
>>> I experienced injustice and the reason of in OSX for it.
>>
>> What was the problem with OSX? Did it throw a warning of you did an
>> equality test on an undefined symbol?
>
> The following if evaluated to true on osx, although I'm pretty sure that
> _MSC_VER isn't defined on osx ;-)
> #if (_MSC_VER < 1300)
> ...
> #endif
>
> replacing it with
> #ifdef WIN32
> #if (_MSC_VER < 1300)
> ...
> #endif
> #endif
>
> fixed the problem.
>
> greetings, Florian Pflug

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Florian G. Pflug 2006-07-27 00:56:02 Re: [HACKERS] Patch for VS.Net 2005's strxfrm() bug
Previous Message andrew 2006-07-27 00:48:11 Re: [HACKERS] Patch for VS.Net 2005's strxfrm() bug

Browse pgsql-patches by date

  From Date Subject
Next Message Florian G. Pflug 2006-07-27 00:56:02 Re: [HACKERS] Patch for VS.Net 2005's strxfrm() bug
Previous Message andrew 2006-07-27 00:48:11 Re: [HACKERS] Patch for VS.Net 2005's strxfrm() bug