On Tue, 2002-05-14 at 04:03, Tom Lane wrote:
> Lamar Owen <lamar.owen@wgcr.org> writes:
> > Although this config file stuff is small potatoes compared to the
> > Win32 stuff as recently discussed. And for that, please understand
> > that most of the developers here consider Win32 an inferior server
> > platform. In fact, Win32 _is_ an inferior server platform, at least
> > in my opinion. But, if you want to do the work, and it doesn't break
> > my non-Win32 server build, by all means go for it.
>
> Note that "doesn't break non-Win32 builds" is not really the standard
> that will get applied. Ongoing readability and maintainability of the
> codebase is a very high priority in my eyes, and I think in the eyes
> of most of the key developers. To the extent that Win32 support can
> be added without hurting those goals, I have nothing against it.
> I'll even put up with localized ugliness (see the BeOS support hacks
> for an example of what I'd call localized ugliness). But I get unhappy
> when there's airy handwaving about moving all static variables into some
> global data structure,
What would your opinion be of some hack with macros, like
#if (Win32 or THREADED)
#define GLOBAL_ pg_globals.
#else
#define GLOBAL_
#endif
and then use global variables as
GLOBAL_globvar
At least in my opinion that would increase both readability and
maintainability.
> to take just one of the points that were under
> discussion last week. That'd be a big maintainability penalty IMHO.
-----------------
Hannu