diff options
| author | Robert Haas <rhaas@postgresql.org> | 2016-12-12 15:57:35 -0500 |
|---|---|---|
| committer | Robert Haas <rhaas@postgresql.org> | 2016-12-12 15:57:35 -0500 |
| commit | 3856cf9607f41245ec9462519c53f1109e781fc5 (patch) | |
| tree | 8614bf59e364edb0cf2736df75597930cd8444fb /src/backend/commands/cluster.c | |
| parent | 9b3d02c2a9eb93cc4754857361abee449a3fe0cb (diff) | |
| download | postgresql-3856cf9607f41245ec9462519c53f1109e781fc5.tar.gz postgresql-3856cf9607f41245ec9462519c53f1109e781fc5.zip | |
Remove should_free arguments to tuplesort routines.
Since commit e94568ecc10f2638e542ae34f2990b821bbf90ac, the answer is
always "false", and we do not need to complicate the API by arranging
to return a constant value.
Peter Geoghegan
Discussion: http://postgr.es/m/CAM3SWZQWZZ_N=DmmL7tKy_OUjGH_5mN=N=A6h7kHyyDvEhg2DA@mail.gmail.com
Diffstat (limited to 'src/backend/commands/cluster.c')
| -rw-r--r-- | src/backend/commands/cluster.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/src/backend/commands/cluster.c b/src/backend/commands/cluster.c index dc1f79f5948..21312268a02 100644 --- a/src/backend/commands/cluster.c +++ b/src/backend/commands/cluster.c @@ -1057,11 +1057,10 @@ copy_heap_data(Oid OIDNewHeap, Oid OIDOldHeap, Oid OIDOldIndex, bool verbose, for (;;) { HeapTuple tuple; - bool shouldfree; CHECK_FOR_INTERRUPTS(); - tuple = tuplesort_getheaptuple(tuplesort, true, &shouldfree); + tuple = tuplesort_getheaptuple(tuplesort, true); if (tuple == NULL) break; @@ -1069,9 +1068,6 @@ copy_heap_data(Oid OIDNewHeap, Oid OIDOldHeap, Oid OIDOldIndex, bool verbose, oldTupDesc, newTupDesc, values, isnull, NewHeap->rd_rel->relhasoids, rwstate); - - if (shouldfree) - heap_freetuple(tuple); } tuplesort_end(tuplesort); |