"Zeugswetter Andreas SB SD" <[EMAIL PROTECTED]> writes:

>   #include <errno.h>
> + #if defined(__CYGWIN__) || defined(WIN32)
>   #include <fcntl.h>                          /* ensure O_BINARY is available */
> + #endif

I don't feel that this will fly.  The comment on the #include line is
hopelessly out of touch with reality --- if you check the list of
symbols defined in fcntl.h you will find a bunch that are used all over
the place, eg SEEK_SET, O_RDONLY, O_CREAT, O_NDELAY, S_IRUSR.  Perhaps
we could #include <fcntl.h> in the various .c files that use these
symbols rather than in c.h, but is that really a step forward?  If there
are any such .c files that also need the conflicting AIX header, we're
still screwed.  Please propose another answer.  Or get AIX to fix their
broken headers.

I have no strong objection to the ecpg part of the proposed patch,
though like Andreas I wonder what's really going on there.  The C99
spec is perfectly clear that LLONG_MIN is the standard spelling ---
why is ecpg apparently choosing another preferred spelling?

                        regards, tom lane

---------------------------(end of broadcast)---------------------------
TIP 9: the planner will ignore your desire to choose an index scan if your
      joining column's datatypes do not match

Reply via email to