On 2014-06-03 10:55:17 -0400, Tom Lane wrote:
> Andres Freund <and...@2ndquadrant.com> writes:
> > I recently had the need to use strtoull() in postgres code. Only to
> > discover that that's not available on some platforms. IIRC windows/msvc
> > was one of them. Now 9d7ded0f4277f5c0063eca8e871a34e2355a8371 added
> > another user - guarded by HAVE_STRTOULL. That commit will make things
> > worse on windows btw...
> Worse than what?  AFAICT, the old code would produce complete garbage
> on Windows.  The new code at least gives the right answer for rowcounts
> up to 4GB.

Hm? Wouldn't it only have produced garbage on mingw and not msvc?

> > How about adding emulation for strtoll/strtoull to port/? The BSDs have
> > easily crib-able functions available...
> Ugh.  Surely Windows has got *some* equivalent, perhaps named differently?

Apparently they've added strtoull()/stroll() to msvc 2013... And there's
_strtoui64() which seems to have already existed a while back.

But it seems easier to me to add one fallback centrally somewhere that
works on all platforms. I am not sure that msvc is the only platform
missing strtoull() - although I didn't find anything relevant in a quick
search through the buildfarm. So maybe I am worrying over nothing.


Andres Freund

 Andres Freund                     http://www.2ndQuadrant.com/
 PostgreSQL Development, 24x7 Support, Training & Services

Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:

Reply via email to