diff options
| author | Michael Paquier <michael@paquier.xyz> | 2024-10-08 10:51:20 +0900 |
|---|---|---|
| committer | Michael Paquier <michael@paquier.xyz> | 2024-10-08 10:51:20 +0900 |
| commit | 4572d59e3c9b0ef6c056544b6336561efd9bb5ec (patch) | |
| tree | f29711312d0bfd8e1eced82822027ca787b6ece3 /src/backend/executor/execGrouping.c | |
| parent | a9ed7d944965865e5c6b44e58602f621cd185856 (diff) | |
Improve style of two code paths
In execGrouping.c, execTuplesMatchPrepare() was doing a memory
allocation that was not necessary when the number of columns was 0.
In foreign.c, pg_options_to_table() was assigning twice a variable to
the same value.
Author: Ranier Vilela
Discussion: https://postgr.es/m/CAEudQAqup0agbSzMjSLSTn=OANyCzxENF1+HrSYnr3WyZib7=Q@mail.gmail.com
Diffstat (limited to 'src/backend/executor/execGrouping.c')
| -rw-r--r-- | src/backend/executor/execGrouping.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/backend/executor/execGrouping.c b/src/backend/executor/execGrouping.c index 7233f1e3c03..774e4de8828 100644 --- a/src/backend/executor/execGrouping.c +++ b/src/backend/executor/execGrouping.c @@ -62,13 +62,15 @@ execTuplesMatchPrepare(TupleDesc desc, const Oid *collations, PlanState *parent) { - Oid *eqFunctions = (Oid *) palloc(numCols * sizeof(Oid)); + Oid *eqFunctions; int i; ExprState *expr; if (numCols == 0) return NULL; + eqFunctions = (Oid *) palloc(numCols * sizeof(Oid)); + /* lookup equality functions */ for (i = 0; i < numCols; i++) eqFunctions[i] = get_opcode(eqOperators[i]); |