On Tue, 2004-11-30 at 23:15 -0500, Tom Lane wrote:
> And how badly does it leak memory? I do not believe this patch is
> tenable.
Did you read the rest of my mail?
> Something that occurred to me the other morning in the shower is that we
> could trivially inline MemoryContextSwitchTo() when using gcc, much as
> you did for list_length(). I haven't gotten around to doing it but it
> seems like a free percent-or-two improvement.
Yeah, it actually occurred to me as well this would be worth doing. It's
not relevant to this particular example though.
-Neil
