pgsql: Remove extra copying of TupleDescs for heap_create_with_catalog

From: Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Remove extra copying of TupleDescs for heap_create_with_catalog
Date: 2011-06-20 14:50:56
Message-ID: E1QYfoW-0002vH-9v@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Remove extra copying of TupleDescs for heap_create_with_catalog

Some callers were creating copies of tuple descriptors to pass to that
function, stating in code comments that it was necessary because it
modified the passed descriptor. Code inspection reveals this not to be
true, and indeed not all callers are passing copies in the first place.
So remove the extra ones and the misleading comments about this behavior
as well.

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/a40a5d9468a5f9f11d355ebf07f7741f5c267588

Modified Files
--------------
src/backend/commands/cluster.c | 9 +++------
src/backend/executor/execMain.c | 8 +-------
2 files changed, 4 insertions(+), 13 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Robert Haas 2011-06-20 16:58:35 Re: Re: [COMMITTERS] pgsql: Fixed string in German translation that causes segfault.
Previous Message Michael Meskes 2011-06-20 13:54:36 Re: Re: [COMMITTERS] pgsql: Fixed string in German translation that causes segfault.