Re: Win64 warnings about size_t - Mailing list pgsql-hackers

From Tom Lane
Subject Re: Win64 warnings about size_t
Date
Msg-id 2406.1262368069@sss.pgh.pa.us
Whole thread Raw
In response to Win64 warnings about size_t  (Magnus Hagander <magnus@hagander.net>)
Responses Re: Win64 warnings about size_t  (Magnus Hagander <magnus@hagander.net>)
List pgsql-hackers
Magnus Hagander <magnus@hagander.net> writes:
> My first 5-6 checks of where these happen are all cases where we
> assign the result of strlen() something to an int, or call a function
> taking an int as parameter with the result of strlen() in there.

Yeah.  Getting rid of all those cases is impractical I think, and
pointless anyway --- we have limitations in palloc and Datum
representation that ensure we'll never be dealing with strings (or other
values) larger than 1GB.

> strlen() returns size_t, which AFAICS is per the standard and not even
> a Microsoft-specific idea.

Correct.

> So I don't really see what win64 does differently in this case, but
> perhaps I've been looking at this code too long? Or is it simply that
> MSVC warns about this and GCC doesn't, and I shuld disbale the
> warning?

I think MSVC is just complaining about something gcc doesn't.  If you
can disable this specific warning it'd be a good plan.
        regards, tom lane


pgsql-hackers by date:

Previous
From: Simon Riggs
Date:
Subject: Re: Cancelling idle in transaction state
Next
From: Magnus Hagander
Date:
Subject: Re: Win64 warnings about size_t