Sync signatures of gbt_var_consistent() and gbt_num_consistent(). For some odd reason we pass the strategy number to gbt_num_consistent as "const StrategyNumber *strategy". There's no reason for that: it almost certainly costs more at both callers and callee to pass a pointer than to pass a small integer value. And it's inconsistent with gbt_var_consistent(), so fix it.
gbt_var_consistent() had its own infelicity, which was not marking the input "key" value const. Fix that too while we're here. This is primarily cosmetic, so I see no need to backpatch. Author: Tom Lane <[email protected]> Discussion: https://postgr.es/m/AH*avqcykhqgvvpwi1giu4oy.8.1781609375063.hmail.3020001...@tju.edu.cn Branch ------ master Details ------- https://git.postgresql.org/pg/commitdiff/4b808ed77cd95dd1d6bf7acdb8ee4f8eb027422c Modified Files -------------- contrib/btree_gist/btree_bool.c | 2 +- contrib/btree_gist/btree_cash.c | 2 +- contrib/btree_gist/btree_date.c | 2 +- contrib/btree_gist/btree_enum.c | 2 +- contrib/btree_gist/btree_float4.c | 2 +- contrib/btree_gist/btree_float8.c | 2 +- contrib/btree_gist/btree_inet.c | 4 ++-- contrib/btree_gist/btree_int2.c | 2 +- contrib/btree_gist/btree_int4.c | 2 +- contrib/btree_gist/btree_int8.c | 2 +- contrib/btree_gist/btree_interval.c | 2 +- contrib/btree_gist/btree_macaddr.c | 2 +- contrib/btree_gist/btree_macaddr8.c | 2 +- contrib/btree_gist/btree_oid.c | 2 +- contrib/btree_gist/btree_time.c | 4 ++-- contrib/btree_gist/btree_ts.c | 4 ++-- contrib/btree_gist/btree_utils_num.c | 4 ++-- contrib/btree_gist/btree_utils_num.h | 2 +- contrib/btree_gist/btree_utils_var.c | 2 +- contrib/btree_gist/btree_utils_var.h | 2 +- contrib/btree_gist/btree_uuid.c | 2 +- 21 files changed, 25 insertions(+), 25 deletions(-)
