From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: [COMMITTERS] pgsql: Modify pg_dump to use error-free memory allocation macros. This |
Date: | 2011-11-26 17:15:39 |
Message-ID: | 201111261715.pAQHFd219184@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
Tom Lane wrote:
> >> It is not possible to just link the old common.c into pg_restore and
> >> pg_dumpall because it contains calls to pg_dump-only functions.
> >>
> >> Ideas?
>
> > The only other idea I have is to keep most functions in the mis-named
> > common.c and move the memory functions into dumpmem.c and dumpmem.h and
> > include that in the other C files, but that doesn't help with long-term
> > code clarity.
>
> I don't think there is any "long-term code clarity" gain here that is
> worth the maintenance headache. Please put the existing functions back
> where they were. Inventing new files for the new code seems fine.
Well, you do a lot more backpatching than I do, so I am willing to do as
you request, but looking at the git history for common.c, I don't see a
lot of backpatching for this file.
Basically, if we keep the existing functions in common.c, we are
deciding that it is never worth moving C functions to new C files for
source code clarity. I was not sure we had made that decision in the
past.
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ It's impossible for everything to be true. +
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Dunstan | 2011-11-26 17:20:20 | pgsql: Ensure plperl strings are always correctly UTF8 encoded. |
Previous Message | Tom Lane | 2011-11-26 15:58:39 | Re: [COMMITTERS] pgsql: Modify pg_dump to use error-free memory allocation macros. This |
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Dunstan | 2011-11-26 17:23:30 | Re: Re: [COMMITTERS] pgsql: Force strings passed to and from plperl to be in UTF8 encoding. |
Previous Message | Robert Haas | 2011-11-26 17:13:47 | Re: Avoiding repeated snapshot computation |