diff options
| author | Tomas Vondra <tomas.vondra@postgresql.org> | 2019-07-05 18:06:02 +0200 |
|---|---|---|
| committer | Tomas Vondra <tomas.vondra@postgresql.org> | 2019-07-05 18:51:56 +0200 |
| commit | ef777cb093e8cb45dd3ae9d3f1499c765147c1dd (patch) | |
| tree | 83e0594ba772cae4f484d6a1e3d9eadb0602c5e8 | |
| parent | 02e95a5049f7933cbde1dacf401604ea3fc02aa5 (diff) | |
| download | postgresql-ef777cb093e8cb45dd3ae9d3f1499c765147c1dd.tar.gz postgresql-ef777cb093e8cb45dd3ae9d3f1499c765147c1dd.zip | |
Remove unused variable in statext_mcv_serialize()
The itemlen variable used to be referenced in multiple places, but since
reworking the serialization code it's used only in one assert. Fixed by
removing the variable and calling the macro from the assert directly.
Backpatch to 12, where this code was introduced.
Reported-by: Jeff Janes
Discussion: https://postgr.es/m/CAMkU=1zc_ovH9NZd_9ovuiEWkF9yX06URUDdXCmgDydf-bqB5A@mail.gmail.com
| -rw-r--r-- | src/backend/statistics/mcv.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/backend/statistics/mcv.c b/src/backend/statistics/mcv.c index 136ebddc460..913a72ff673 100644 --- a/src/backend/statistics/mcv.c +++ b/src/backend/statistics/mcv.c @@ -911,10 +911,9 @@ statext_mcv_serialize(MCVList *mcvlist, VacAttrStats **stats) for (i = 0; i < mcvlist->nitems; i++) { MCVItem *mcvitem = &mcvlist->items[i]; - int itemlen = ITEM_SIZE(dim); /* don't write beyond the allocated space */ - Assert(ptr <= (endptr - itemlen)); + Assert(ptr <= (endptr - ITEM_SIZE(dim))); /* copy NULL and frequency flags into the serialized MCV */ memcpy(ptr, mcvitem->isnull, sizeof(bool) * ndims); |