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/foreign/foreign.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/foreign/foreign.c')
| -rw-r--r-- | src/backend/foreign/foreign.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/backend/foreign/foreign.c b/src/backend/foreign/foreign.c index bb67d9f92b8..4c06e1ff1c4 100644 --- a/src/backend/foreign/foreign.c +++ b/src/backend/foreign/foreign.c @@ -524,7 +524,7 @@ pg_options_to_table(PG_FUNCTION_ARGS) Datum array = PG_GETARG_DATUM(0); ListCell *cell; List *options; - ReturnSetInfo *rsinfo = (ReturnSetInfo *) fcinfo->resultinfo; + ReturnSetInfo *rsinfo; options = untransformRelOptions(array); rsinfo = (ReturnSetInfo *) fcinfo->resultinfo; |