Re: Fairly serious bug induced by latest guc enum changes

Magnus Hagander <magnus@[EMAIL PROTECTED]
> writes:
> Since it didn't really sound like a nice option, here's a third one I 
> came up with later. Basically, this one splits things apart so we only 
> use one variable, which is sync_method. Instead of using a macro to get 
> the open sync bit, it uses a function. This makes the assign hook only 
> responsible for flu****ng and closing the old file.

Okay, but you failed to correctly reproduce the conditions for closing
the old file.

> Thoughts? And if you like it, is it enough to expect the compiler to 
> figure out to inline it or should we explicitly inline it?

I don't think we care that much, since it's only invoked when we're
about to do a moderately expensive kernel call.

			regards, tom lane

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