diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2023-03-02 12:01:47 -0500 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2023-03-02 12:01:47 -0500 |
| commit | 6b661b01f48bb0d3129ad0e3909210a6ba0534b3 (patch) | |
| tree | b44edf0dec23e25d3c332a47123fe538d610ded4 /src/backend/executor/execUtils.c | |
| parent | 00b41463c21615f9bf3927f207e37f9e215d32e6 (diff) | |
| download | postgresql-6b661b01f48bb0d3129ad0e3909210a6ba0534b3.tar.gz postgresql-6b661b01f48bb0d3129ad0e3909210a6ba0534b3.zip | |
Remove local optimizations of empty Bitmapsets into null pointers.
These are all dead code now that it's done centrally.
Patch by me; thanks to Nathan Bossart and Richard Guo for review.
Discussion: https://postgr.es/m/1159933.1677621588@sss.pgh.pa.us
Diffstat (limited to 'src/backend/executor/execUtils.c')
| -rw-r--r-- | src/backend/executor/execUtils.c | 10 |
1 files changed, 1 insertions, 9 deletions
diff --git a/src/backend/executor/execUtils.c b/src/backend/executor/execUtils.c index c33a3c0bec7..cfa95a07e40 100644 --- a/src/backend/executor/execUtils.c +++ b/src/backend/executor/execUtils.c @@ -877,15 +877,7 @@ UpdateChangedParamSet(PlanState *node, Bitmapset *newchg) * include anything else into its chgParam set. */ parmset = bms_intersect(node->plan->allParam, newchg); - - /* - * Keep node->chgParam == NULL if there's not actually any members; this - * allows the simplest possible tests in executor node files. - */ - if (!bms_is_empty(parmset)) - node->chgParam = bms_join(node->chgParam, parmset); - else - bms_free(parmset); + node->chgParam = bms_join(node->chgParam, parmset); } /* |