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-27 03:38:04 |
Message-ID: | 201111270338.pAR3c4Z02328@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
Tom Lane wrote:
> Bruce Momjian <bruce(at)momjian(dot)us> writes:
> > 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.
>
> I once held hope that git would let us be more flexible about this,
> but experimentation has shown that it's not significantly smarter than
> CVS about back-patches in moved code :-(.
>
> I don't want to say that we should *never* move code once it's released;
> but I think the bar for that needs to be set pretty high, and this
> particular case isn't clearing the bar for me.
OK, hearing no other opinions, I have moved the functions back into
common.c, and created a new dumpmem.c/h. Applied patch attached.
There is now no need to modify MSVC.
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ It's impossible for everything to be true. +
Attachment | Content-Type | Size |
---|---|---|
/rtmp/pg_dump | text/x-diff | 64.7 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Dunstan | 2011-11-27 06:23:56 | pgsql: Fix MSVC builds broken by xsubpp change |
Previous Message | Bruce Momjian | 2011-11-27 03:36:54 | pgsql: Move pg_dump memory routines into pg_dumpmem.c/h and restore com |
From | Date | Subject | |
---|---|---|---|
Next Message | NISHIYAMA Tomoaki | 2011-11-27 08:02:40 | Re: [PATCH] PostgreSQL fails to build with 32bit MinGW-w64 |
Previous Message | Bruce Momjian | 2011-11-27 03:36:54 | pgsql: Move pg_dump memory routines into pg_dumpmem.c/h and restore com |