diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2016-09-22 14:30:33 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2016-09-22 14:30:33 -0400 |
| commit | 8023b5827fbada6815ce269db4f3373ac77ec7c3 (patch) | |
| tree | 23d92d4b750de0095e609ef8e75280a4c1536e66 /src/backend/executor/nodeTidscan.c | |
| parent | 96dd77d349424f270d129f8f40e75f762ddcca7d (diff) | |
| download | postgresql-8023b5827fbada6815ce269db4f3373ac77ec7c3.tar.gz postgresql-8023b5827fbada6815ce269db4f3373ac77ec7c3.zip | |
Remove nearly-unused SizeOfIptrData macro.
Past refactorings have removed all but one reference to SizeOfIptrData
(and that one place was in a pretty noncritical spot). Since nobody's
complained, it seems probable that there are no supported compilers
that don't think sizeof(ItemPointerData) is 6. If there are, we're
wasting MAXALIGN per heap tuple anyway, so it's rather silly to worry
about whether we can shave space in places like WAL records.
Pavan Deolasee
Discussion: <CABOikdOOawDda4hwLOT6zdA6MFfPLu3Z2YBZkX0JdayNS6JOeQ@mail.gmail.com>
Diffstat (limited to 'src/backend/executor/nodeTidscan.c')
| -rw-r--r-- | src/backend/executor/nodeTidscan.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/backend/executor/nodeTidscan.c b/src/backend/executor/nodeTidscan.c index 26041033529..d54fe3665f7 100644 --- a/src/backend/executor/nodeTidscan.c +++ b/src/backend/executor/nodeTidscan.c @@ -139,7 +139,7 @@ TidListCreate(TidScanState *tidstate) continue; itemarray = DatumGetArrayTypeP(arraydatum); deconstruct_array(itemarray, - TIDOID, SizeOfIptrData, false, 's', + TIDOID, sizeof(ItemPointerData), false, 's', &ipdatums, &ipnulls, &ndatums); if (numTids + ndatums > numAllocTids) { |