From 33a3b03d636b529b27f869e332b6344d52635331 Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Fri, 20 Feb 2015 17:32:01 -0500 Subject: Use FLEXIBLE_ARRAY_MEMBER in some more places. Fix a batch of structs that are only visible within individual .c files. Michael Paquier --- src/backend/executor/nodeAgg.c | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) (limited to 'src/backend/executor') diff --git a/src/backend/executor/nodeAgg.c b/src/backend/executor/nodeAgg.c index 8079d977643..9ff0eff6d7b 100644 --- a/src/backend/executor/nodeAgg.c +++ b/src/backend/executor/nodeAgg.c @@ -297,9 +297,9 @@ typedef struct AggHashEntryData *AggHashEntry; typedef struct AggHashEntryData { TupleHashEntryData shared; /* common header for hash table entries */ - /* per-aggregate transition status array - must be last! */ - AggStatePerGroupData pergroup[1]; /* VARIABLE LENGTH ARRAY */ -} AggHashEntryData; /* VARIABLE LENGTH STRUCT */ + /* per-aggregate transition status array */ + AggStatePerGroupData pergroup[FLEXIBLE_ARRAY_MEMBER]; +} AggHashEntryData; static void initialize_aggregates(AggState *aggstate, @@ -941,8 +941,8 @@ build_hash_table(AggState *aggstate) Assert(node->aggstrategy == AGG_HASHED); Assert(node->numGroups > 0); - entrysize = sizeof(AggHashEntryData) + - (aggstate->numaggs - 1) * sizeof(AggStatePerGroupData); + entrysize = offsetof(AggHashEntryData, pergroup) + + aggstate->numaggs * sizeof(AggStatePerGroupData); aggstate->hashtable = BuildTupleHashTable(node->numCols, node->grpColIdx, @@ -1013,8 +1013,8 @@ hash_agg_entry_size(int numAggs) Size entrysize; /* This must match build_hash_table */ - entrysize = sizeof(AggHashEntryData) + - (numAggs - 1) * sizeof(AggStatePerGroupData); + entrysize = offsetof(AggHashEntryData, pergroup) + + numAggs * sizeof(AggStatePerGroupData); entrysize = MAXALIGN(entrysize); /* Account for hashtable overhead (assuming fill factor = 1) */ entrysize += 3 * sizeof(void *); -- cgit v1.2.3