> A pg_dump run is comparatively short-lived, so if Zdenek is right then
> there's no important leak here -- we're counting on program exit to
> release the memory. There's probably little point in releasing things
> earlier than that.
Well, I'd tend to consider any logical part of a program that fails to
release the memory it uses to be bad coding practice. You never know
when you're going to need to shuffle things around, change the context
of the code in a way that makes it long-lived, in turn causing the leak
to become a real problem. Also, don't you like seeing the free()s paired
to their mallocs()s in a way that makes the allocations intuitively
correct? :)
--
Tomáš Szépe <szepe@pinerecords.com>