Remove duplicate lines of code
authorDaniel Gustafsson
Mon, 24 Apr 2023 09:16:17 +0000 (11:16 +0200)
committerDaniel Gustafsson
Mon, 24 Apr 2023 09:16:17 +0000 (11:16 +0200)
Commit 6df7a9698bb accidentally included two identical prototypes for
default_multirange_selectivi() and commit 086cf1458c6 added a break;
statement where one was already present, thus duplicating it.  While
there is no bug caused by this, fix by removing the duplicated lines
as they provide no value.

Backpatch the fix for duplicate prototypes to v14 and the duplicate
break statement fix to all supported branches to avoid backpatching
hazards due to the removal.

Reported-by: Anton Voloshin
Discussion: https://postgr.es/m/0e69cb60-0176-f6d0-7e15-6478b7d85724@postgrespro.ru

src/backend/utils/adt/multirangetypes_selfuncs.c
src/interfaces/ecpg/preproc/variable.c

index 8d085ca8d4fca0c13c44096f4628e8436a12c796..0360a17d8c6343b748f897c1950e319c6da3a740 100644 (file)
@@ -35,7 +35,6 @@ static double calc_multirangesel(TypeCacheEntry *typcache,
                                 VariableStatData *vardata,
                                 const MultirangeType *constval, Oid operator);
 static double default_multirange_selectivity(Oid operator);
-static double default_multirange_selectivity(Oid operator);
 static double calc_hist_selectivity(TypeCacheEntry *typcache,
                                    VariableStatData *vardata,
                                    const MultirangeType *constval,
index 887d479e73518c538a550e9fb13a06bfd022b206..8926676ab7178bd80fdefb9ccd6fdfef5314b983 100644 (file)
@@ -105,7 +105,6 @@ find_struct_member(char *name, char *str, struct ECPGstruct_member *members, int
                        else
                            return find_struct_member(name, ++end, members->type->u.members, brace_level);
                        break;
-                       break;
                    case '.':
                        if (members->type->type == ECPGt_array)
                            return find_struct_member(name, end, members->type->u.element->u.members, brace_level);