Andreas Pflug <[EMAIL PROTECTED]> writes:
> <sys/time.h> in libpq-be.h isn't present on win32 vc6. Apparently, it 
> isn't used in Linux either; libpq will compile without it.

It's there to declare struct timeval, and I'm fairly certain that diking
it out of the header would break things on some platforms.  Where does
Windows define struct timeval?
> +#ifndef WIN32
>  #include <sys/time.h>
> +#endif

Could we please use HAVE_SYS_TIME_H, rather than creating an unnecessary
platform specificity?

>  #ifndef ELOG_H
>  #define ELOG_H
> +#ifdef ERROR
> +#undef ERROR    /* possible conflict in win32 */
> +#endif
> +
>  /* Error level codes */

This seems likely to break anything expecting the Win32 definition
of ERROR.  Can we look at not including elog.h in frontend builds,
instead?  I can't think of any reason for frontend code to need it.

                        regards, tom lane

---------------------------(end of broadcast)---------------------------
TIP 1: subscribe and unsubscribe commands go to [EMAIL PROTECTED]

Reply via email to