From: | Magnus Hagander <magnus(at)hagander(dot)net> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | pgsql-patches(at)postgresql(dot)org |
Subject: | Re: Fix for win32 stat() problems |
Date: | 2008-04-10 15:16:01 |
Message-ID: | 20080410171601.7b54ca42@mha-laptop |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-patches |
Tom Lane wrote:
> Magnus Hagander <magnus(at)hagander(dot)net> writes:
> > A whole lot simpler patch :-)
>
> Seems like you no longer need the !defined(_DIRMOD_C) bit here?
Correct. That wasn't the actual error, that was me misdiagnosing the
situation.
> Also please include a comment about why <sys/stat.h> has to be
> forcibly included.
Will do.
> A more general question: can't we get rid of most of the #ifdef WIN32
> cruft in include/port.h, and put it in include/port/win32.h instead?
> Seems cleaner that way, at least for things where there's just an
> #ifdef WIN32 hunk and not two cases for Win and not-Win.
Yeah, that one has been on my TODO for a while.
We may not be able to move everything because one file is included
early in the pass of c.h and one much later, but the majority of the
stuff we have there shouldn't care about which order it goes in at.
//Magnus
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Chernow | 2008-04-10 15:16:37 | Re: [PATCHES] libpq type system 0.9a |
Previous Message | Merlin Moncure | 2008-04-10 15:14:45 | Re: [PATCHES] libpq type system 0.9a |