Skip to content

Commit eccb607

Browse files
Fix recent cpluspluscheck issue in selfuncs.h.
Fix selfuncs.h cpluspluscheck complaint, without reintroducing a parameter name inconsistency (restore the original declaration names, and then make corresponding function definitions consistent with that). Oversight in commit a601366. Author: Peter Geoghegan <[email protected]> Reported-By: Andres Freund <[email protected]>
1 parent a601366 commit eccb607

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

src/backend/utils/adt/selfuncs.c

+3-3
Original file line numberDiff line numberDiff line change
@@ -290,7 +290,7 @@ eqsel_internal(PG_FUNCTION_ARGS, bool negate)
290290
* This is exported so that some other estimation functions can use it.
291291
*/
292292
double
293-
var_eq_const(VariableStatData *vardata, Oid operator, Oid collation,
293+
var_eq_const(VariableStatData *vardata, Oid oproid, Oid collation,
294294
Datum constval, bool constisnull,
295295
bool varonleft, bool negate)
296296
{
@@ -331,7 +331,7 @@ var_eq_const(VariableStatData *vardata, Oid operator, Oid collation,
331331
}
332332
else if (HeapTupleIsValid(vardata->statsTuple) &&
333333
statistic_proc_security_check(vardata,
334-
(opfuncoid = get_opcode(operator))))
334+
(opfuncoid = get_opcode(oproid))))
335335
{
336336
AttStatsSlot sslot;
337337
bool match = false;
@@ -461,7 +461,7 @@ var_eq_const(VariableStatData *vardata, Oid operator, Oid collation,
461461
* This is exported so that some other estimation functions can use it.
462462
*/
463463
double
464-
var_eq_non_const(VariableStatData *vardata, Oid operator, Oid collation,
464+
var_eq_non_const(VariableStatData *vardata, Oid oproid, Oid collation,
465465
Node *other,
466466
bool varonleft, bool negate)
467467
{

src/include/utils/selfuncs.h

+2-2
Original file line numberDiff line numberDiff line change
@@ -181,11 +181,11 @@ extern double ineq_histogram_selectivity(PlannerInfo *root,
181181
Oid collation,
182182
Datum constval, Oid consttype);
183183
extern double var_eq_const(VariableStatData *vardata,
184-
Oid operator, Oid collation,
184+
Oid oproid, Oid collation,
185185
Datum constval, bool constisnull,
186186
bool varonleft, bool negate);
187187
extern double var_eq_non_const(VariableStatData *vardata,
188-
Oid operator, Oid collation,
188+
Oid oproid, Oid collation,
189189
Node *other,
190190
bool varonleft, bool negate);
191191

0 commit comments

Comments
 (0)