pg_dump warnings in MinGW build

Started by Jeff Janesalmost 12 years ago2 messageshackers
Jump to latest
#1Jeff Janes
jeff.janes@gmail.com

Now that popen and pclose don't throw thousands of warnings when compiling
mingw builds, some other warnings stand out.

parallel.c: In function 'pgpipe':
parallel.c:1332:2: warning: overflow in implicit constant conversion
[-Woverflow]
parallel.c:1386:3: warning: overflow in implicit constant conversion
[-Woverflow]

I think the solution is to use the pgsocket typedef from
src/include/port.h, rather than int. Like attached.

But I'm far from being a typedef lawyer, so maybe I am all wet.

Cheers,

Jeff

Attachments:

pgpipe.patchapplication/octet-stream; name=pgpipe.patchDownload+3-3
#2Bruce Momjian
bruce@momjian.us
In reply to: Jeff Janes (#1)
Re: pg_dump warnings in MinGW build

On Thu, May 15, 2014 at 10:52:43AM -0700, Jeff Janes wrote:

Now that popen and pclose don't throw thousands of warnings when compiling
mingw builds, some other warnings stand out.

parallel.c: In function 'pgpipe':
parallel.c:1332:2: warning: overflow in implicit constant conversion
[-Woverflow]
parallel.c:1386:3: warning: overflow in implicit constant conversion
[-Woverflow]

I think the solution is to use the pgsocket typedef from src/include/port.h,
rather than int. �Like attached.

But I'm far from being a typedef lawyer, so maybe I am all wet.

FYI, I am not sure if you saw that this was fixed in June as part of a
larger patch:

commit ac608fe758455804f26179ea7c556e7752e453e8
Author: Bruce Momjian <bruce@momjian.us>
Date: Mon Jun 16 15:24:38 2014 -0400

Use type pgsocket for Windows pipe emulation socket calls

This prevents several compiler warnings on Windows.

--
Bruce Momjian <bruce@momjian.us> http://momjian.us
EnterpriseDB http://enterprisedb.com

+ Everyone has their own god. +

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