Skip to content

Commit 1356f78

Browse files
peterepetere
committed
Reduce excessive dereferencing of function pointers
It is equivalent in ANSI C to write (*funcptr) () and funcptr(). These two styles have been applied inconsistently. After discussion, we'll use the more verbose style for plain function pointer variables, to make it clear that it's a variable, and the shorter style when the function pointer is in a struct (s.func() or s->func()), because then it's clear that it's not a plain function name, and otherwise the excessive punctuation makes some of those invocations hard to read. Discussion: https://www.postgresql.org/message-id/f52c16db-14ed-757d-4b48-7ef360b1631d@2ndquadrant.com
1 parent 9d71323 commit 1356f78

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

46 files changed

+249
-250
lines changed

‎contrib/btree_gist/btree_utils_num.c

Lines changed: 17 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -184,10 +184,10 @@ gbt_num_union(GBT_NUMKEY *out, const GistEntryVector *entryvec, const gbtree_nin
184184
c.lower = &cur[0];
185185
c.upper = &cur[tinfo->size];
186186
/* if out->lower > cur->lower, adopt cur as lower */
187-
if ((*tinfo->f_gt) (o.lower, c.lower, flinfo))
187+
if (tinfo->f_gt(o.lower, c.lower, flinfo))
188188
memcpy((void *) o.lower, (void *) c.lower, tinfo->size);
189189
/* if out->upper < cur->upper, adopt cur as upper */
190-
if ((*tinfo->f_lt) (o.upper, c.upper, flinfo))
190+
if (tinfo->f_lt(o.upper, c.upper, flinfo))
191191
memcpy((void *) o.upper, (void *) c.upper, tinfo->size);
192192
}
193193

@@ -211,8 +211,8 @@ gbt_num_same(const GBT_NUMKEY *a, const GBT_NUMKEY *b, const gbtree_ninfo *tinfo
211211
b2.lower = &(((GBT_NUMKEY *) b)[0]);
212212
b2.upper = &(((GBT_NUMKEY *) b)[tinfo->size]);
213213

214-
return ((*tinfo->f_eq) (b1.lower, b2.lower, flinfo) &&
215-
(*tinfo->f_eq) (b1.upper, b2.upper, flinfo));
214+
return (tinfo->f_eq(b1.lower, b2.lower, flinfo) &&
215+
tinfo->f_eq(b1.upper, b2.upper, flinfo));
216216
}
217217

218218

@@ -236,9 +236,9 @@ gbt_num_bin_union(Datum *u, GBT_NUMKEY *e, const gbtree_ninfo *tinfo, FmgrInfo *
236236

237237
ur.lower = &(((GBT_NUMKEY *) DatumGetPointer(*u))[0]);
238238
ur.upper = &(((GBT_NUMKEY *) DatumGetPointer(*u))[tinfo->size]);
239-
if ((*tinfo->f_gt) ((void *) ur.lower, (void *) rd.lower, flinfo))
239+
if (tinfo->f_gt((void *) ur.lower, (void *) rd.lower, flinfo))
240240
memcpy((void *) ur.lower, (void *) rd.lower, tinfo->size);
241-
if ((*tinfo->f_lt) ((void *) ur.upper, (void *) rd.upper, flinfo))
241+
if (tinfo->f_lt((void *) ur.upper, (void *) rd.upper, flinfo))
242242
memcpy((void *) ur.upper, (void *) rd.upper, tinfo->size);
243243
}
244244
}
@@ -264,33 +264,33 @@ gbt_num_consistent(const GBT_NUMKEY_R *key,
264264
switch (*strategy)
265265
{
266266
case BTLessEqualStrategyNumber:
267-
retval = (*tinfo->f_ge) (query, key->lower, flinfo);
267+
retval = tinfo->f_ge(query, key->lower, flinfo);
268268
break;
269269
case BTLessStrategyNumber:
270270
if (is_leaf)
271-
retval = (*tinfo->f_gt) (query, key->lower, flinfo);
271+
retval = tinfo->f_gt(query, key->lower, flinfo);
272272
else
273-
retval = (*tinfo->f_ge) (query, key->lower, flinfo);
273+
retval = tinfo->f_ge(query, key->lower, flinfo);
274274
break;
275275
case BTEqualStrategyNumber:
276276
if (is_leaf)
277-
retval = (*tinfo->f_eq) (query, key->lower, flinfo);
277+
retval = tinfo->f_eq(query, key->lower, flinfo);
278278
else
279-
retval = ((*tinfo->f_le) (key->lower, query, flinfo) &&
280-
(*tinfo->f_le) (query, key->upper, flinfo));
279+
retval = (tinfo->f_le(key->lower, query, flinfo) &&
280+
tinfo->f_le(query, key->upper, flinfo));
281281
break;
282282
case BTGreaterStrategyNumber:
283283
if (is_leaf)
284-
retval = (*tinfo->f_lt) (query, key->upper, flinfo);
284+
retval = tinfo->f_lt(query, key->upper, flinfo);
285285
else
286-
retval = (*tinfo->f_le) (query, key->upper, flinfo);
286+
retval = tinfo->f_le(query, key->upper, flinfo);
287287
break;
288288
case BTGreaterEqualStrategyNumber:
289-
retval = (*tinfo->f_le) (query, key->upper, flinfo);
289+
retval = tinfo->f_le(query, key->upper, flinfo);
290290
break;
291291
case BtreeGistNotEqualStrategyNumber:
292-
retval = (!((*tinfo->f_eq) (query, key->lower, flinfo) &&
293-
(*tinfo->f_eq) (query, key->upper, flinfo)));
292+
retval = (!(tinfo->f_eq(query, key->lower, flinfo) &&
293+
tinfo->f_eq(query, key->upper, flinfo)));
294294
break;
295295
default:
296296
retval = false;

‎contrib/btree_gist/btree_utils_var.c

Lines changed: 22 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,7 @@ gbt_var_leaf2node(GBT_VARKEY *leaf, const gbtree_vinfo *tinfo, FmgrInfo *flinfo)
109109
GBT_VARKEY *out = leaf;
110110

111111
if (tinfo->f_l2n)
112-
out = (*tinfo->f_l2n) (leaf, flinfo);
112+
out = tinfo->f_l2n(leaf, flinfo);
113113

114114
return out;
115115
}
@@ -255,13 +255,13 @@ gbt_var_bin_union(Datum *u, GBT_VARKEY *e, Oid collation,
255255
nr.lower = ro.lower;
256256
nr.upper = ro.upper;
257257

258-
if ((*tinfo->f_cmp) (ro.lower, eo.lower, collation, flinfo) > 0)
258+
if (tinfo->f_cmp(ro.lower, eo.lower, collation, flinfo) > 0)
259259
{
260260
nr.lower = eo.lower;
261261
update = true;
262262
}
263263

264-
if ((*tinfo->f_cmp) (ro.upper, eo.upper, collation, flinfo) < 0)
264+
if (tinfo->f_cmp(ro.upper, eo.upper, collation, flinfo) < 0)
265265
{
266266
nr.upper = eo.upper;
267267
update = true;
@@ -371,8 +371,8 @@ gbt_var_same(Datum d1, Datum d2, Oid collation,
371371
r1 = gbt_var_key_readable(t1);
372372
r2 = gbt_var_key_readable(t2);
373373

374-
return ((*tinfo->f_cmp) (r1.lower, r2.lower, collation, flinfo) == 0 &&
375-
(*tinfo->f_cmp) (r1.upper, r2.upper, collation, flinfo) == 0);
374+
return (tinfo->f_cmp(r1.lower, r2.lower, collation, flinfo) == 0 &&
375+
tinfo->f_cmp(r1.upper, r2.upper, collation, flinfo) == 0);
376376
}
377377

378378

@@ -400,9 +400,9 @@ gbt_var_penalty(float *res, const GISTENTRY *o, const GISTENTRY *n,
400400

401401
if ((VARSIZE(ok.lower) - VARHDRSZ) == 0 && (VARSIZE(ok.upper) - VARHDRSZ) == 0)
402402
*res = 0.0;
403-
else if (!(((*tinfo->f_cmp) (nk.lower, ok.lower, collation, flinfo) >= 0 ||
403+
else if (!((tinfo->f_cmp(nk.lower, ok.lower, collation, flinfo) >= 0 ||
404404
gbt_bytea_pf_match(ok.lower, nk.lower, tinfo)) &&
405-
((*tinfo->f_cmp) (nk.upper, ok.upper, collation, flinfo) <= 0 ||
405+
(tinfo->f_cmp(nk.upper, ok.upper, collation, flinfo) <= 0 ||
406406
gbt_bytea_pf_match(ok.upper, nk.upper, tinfo))))
407407
{
408408
Datum d = PointerGetDatum(0);
@@ -449,9 +449,9 @@ gbt_vsrt_cmp(const void *a, const void *b, void *arg)
449449
const gbt_vsrt_arg *varg = (const gbt_vsrt_arg *) arg;
450450
int res;
451451

452-
res = (*varg->tinfo->f_cmp) (ar.lower, br.lower, varg->collation, varg->flinfo);
452+
res = varg->tinfo->f_cmp(ar.lower, br.lower, varg->collation, varg->flinfo);
453453
if (res == 0)
454-
return (*varg->tinfo->f_cmp) (ar.upper, br.upper, varg->collation, varg->flinfo);
454+
return varg->tinfo->f_cmp(ar.upper, br.upper, varg->collation, varg->flinfo);
455455

456456
return res;
457457
}
@@ -567,44 +567,44 @@ gbt_var_consistent(GBT_VARKEY_R *key,
567567
{
568568
case BTLessEqualStrategyNumber:
569569
if (is_leaf)
570-
retval = (*tinfo->f_ge) (query, key->lower, collation, flinfo);
570+
retval = tinfo->f_ge(query, key->lower, collation, flinfo);
571571
else
572-
retval = (*tinfo->f_cmp) (query, key->lower, collation, flinfo) >= 0
572+
retval = tinfo->f_cmp(query, key->lower, collation, flinfo) >= 0
573573
|| gbt_var_node_pf_match(key, query, tinfo);
574574
break;
575575
case BTLessStrategyNumber:
576576
if (is_leaf)
577-
retval = (*tinfo->f_gt) (query, key->lower, collation, flinfo);
577+
retval = tinfo->f_gt(query, key->lower, collation, flinfo);
578578
else
579-
retval = (*tinfo->f_cmp) (query, key->lower, collation, flinfo) >= 0
579+
retval = tinfo->f_cmp(query, key->lower, collation, flinfo) >= 0
580580
|| gbt_var_node_pf_match(key, query, tinfo);
581581
break;
582582
case BTEqualStrategyNumber:
583583
if (is_leaf)
584-
retval = (*tinfo->f_eq) (query, key->lower, collation, flinfo);
584+
retval = tinfo->f_eq(query, key->lower, collation, flinfo);
585585
else
586586
retval =
587-
((*tinfo->f_cmp) (key->lower, query, collation, flinfo) <= 0 &&
588-
(*tinfo->f_cmp) (query, key->upper, collation, flinfo) <= 0) ||
587+
(tinfo->f_cmp(key->lower, query, collation, flinfo) <= 0 &&
588+
tinfo->f_cmp(query, key->upper, collation, flinfo) <= 0) ||
589589
gbt_var_node_pf_match(key, query, tinfo);
590590
break;
591591
case BTGreaterStrategyNumber:
592592
if (is_leaf)
593-
retval = (*tinfo->f_lt) (query, key->upper, collation, flinfo);
593+
retval = tinfo->f_lt(query, key->upper, collation, flinfo);
594594
else
595-
retval = (*tinfo->f_cmp) (query, key->upper, collation, flinfo) <= 0
595+
retval = tinfo->f_cmp(query, key->upper, collation, flinfo) <= 0
596596
|| gbt_var_node_pf_match(key, query, tinfo);
597597
break;
598598
case BTGreaterEqualStrategyNumber:
599599
if (is_leaf)
600-
retval = (*tinfo->f_le) (query, key->upper, collation, flinfo);
600+
retval = tinfo->f_le(query, key->upper, collation, flinfo);
601601
else
602-
retval = (*tinfo->f_cmp) (query, key->upper, collation, flinfo) <= 0
602+
retval = tinfo->f_cmp(query, key->upper, collation, flinfo) <= 0
603603
|| gbt_var_node_pf_match(key, query, tinfo);
604604
break;
605605
case BtreeGistNotEqualStrategyNumber:
606-
retval = !((*tinfo->f_eq) (query, key->lower, collation, flinfo) &&
607-
(*tinfo->f_eq) (query, key->upper, collation, flinfo));
606+
retval = !(tinfo->f_eq(query, key->lower, collation, flinfo) &&
607+
tinfo->f_eq(query, key->upper, collation, flinfo));
608608
break;
609609
default:
610610
retval = FALSE;

‎src/backend/access/transam/xact.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3347,7 +3347,7 @@ CallXactCallbacks(XactEvent event)
33473347
XactCallbackItem *item;
33483348

33493349
for (item = Xact_callbacks; item; item = item->next)
3350-
(*item->callback) (event, item->arg);
3350+
item->callback(event, item->arg);
33513351
}
33523352

33533353

@@ -3404,7 +3404,7 @@ CallSubXactCallbacks(SubXactEvent event,
34043404
SubXactCallbackItem *item;
34053405

34063406
for (item = SubXact_callbacks; item; item = item->next)
3407-
(*item->callback) (event, mySubid, parentSubid, item->arg);
3407+
item->callback(event, mySubid, parentSubid, item->arg);
34083408
}
34093409

34103410

‎src/backend/commands/analyze.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -526,7 +526,7 @@ do_analyze_rel(Relation onerel, int options, VacuumParams *params,
526526

527527
stats->rows = rows;
528528
stats->tupDesc = onerel->rd_att;
529-
(*stats->compute_stats) (stats,
529+
stats->compute_stats(stats,
530530
std_fetch_func,
531531
numrows,
532532
totalrows);
@@ -830,7 +830,7 @@ compute_index_stats(Relation onerel, double totalrows,
830830
stats->exprvals = exprvals + i;
831831
stats->exprnulls = exprnulls + i;
832832
stats->rowstride = attr_cnt;
833-
(*stats->compute_stats) (stats,
833+
stats->compute_stats(stats,
834834
ind_fetch_func,
835835
numindexrows,
836836
totalindexrows);

‎src/backend/commands/portalcmds.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -397,7 +397,7 @@ PersistHoldablePortal(Portal portal)
397397
/* Fetch the result set into the tuplestore */
398398
ExecutorRun(queryDesc, ForwardScanDirection, 0L, false);
399399

400-
(*queryDesc->dest->rDestroy) (queryDesc->dest);
400+
queryDesc->dest->rDestroy(queryDesc->dest);
401401
queryDesc->dest = NULL;
402402

403403
/*

‎src/backend/commands/seclabel.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -122,7 +122,7 @@ ExecSecLabelStmt(SecLabelStmt *stmt)
122122
}
123123

124124
/* Provider gets control here, may throw ERROR to veto new label. */
125-
(*provider->hook) (&address, stmt->label);
125+
provider->hook(&address, stmt->label);
126126

127127
/* Apply new label. */
128128
SetSecurityLabel(&address, provider->provider_name, stmt->label);

‎src/backend/executor/execCurrent.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -220,7 +220,7 @@ fetch_cursor_param_value(ExprContext *econtext, int paramId)
220220

221221
/* give hook a chance in case parameter is dynamic */
222222
if (!OidIsValid(prm->ptype) && paramInfo->paramFetch != NULL)
223-
(*paramInfo->paramFetch) (paramInfo, paramId);
223+
paramInfo->paramFetch(paramInfo, paramId);
224224

225225
if (OidIsValid(prm->ptype) && !prm->isnull)
226226
{

‎src/backend/executor/execExprInterp.c

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -647,7 +647,7 @@ ExecInterpExpr(ExprState *state, ExprContext *econtext, bool *isnull)
647647
FunctionCallInfo fcinfo = op->d.func.fcinfo_data;
648648

649649
fcinfo->isnull = false;
650-
*op->resvalue = (op->d.func.fn_addr) (fcinfo);
650+
*op->resvalue = op->d.func.fn_addr(fcinfo);
651651
*op->resnull = fcinfo->isnull;
652652

653653
EEO_NEXT();
@@ -669,7 +669,7 @@ ExecInterpExpr(ExprState *state, ExprContext *econtext, bool *isnull)
669669
}
670670
}
671671
fcinfo->isnull = false;
672-
*op->resvalue = (op->d.func.fn_addr) (fcinfo);
672+
*op->resvalue = op->d.func.fn_addr(fcinfo);
673673
*op->resnull = fcinfo->isnull;
674674

675675
strictfail:
@@ -684,7 +684,7 @@ ExecInterpExpr(ExprState *state, ExprContext *econtext, bool *isnull)
684684
pgstat_init_function_usage(fcinfo, &fcusage);
685685

686686
fcinfo->isnull = false;
687-
*op->resvalue = (op->d.func.fn_addr) (fcinfo);
687+
*op->resvalue = op->d.func.fn_addr(fcinfo);
688688
*op->resnull = fcinfo->isnull;
689689

690690
pgstat_end_function_usage(&fcusage, true);
@@ -712,7 +712,7 @@ ExecInterpExpr(ExprState *state, ExprContext *econtext, bool *isnull)
712712
pgstat_init_function_usage(fcinfo, &fcusage);
713713

714714
fcinfo->isnull = false;
715-
*op->resvalue = (op->d.func.fn_addr) (fcinfo);
715+
*op->resvalue = op->d.func.fn_addr(fcinfo);
716716
*op->resnull = fcinfo->isnull;
717717

718718
pgstat_end_function_usage(&fcusage, true);
@@ -1170,7 +1170,7 @@ ExecInterpExpr(ExprState *state, ExprContext *econtext, bool *isnull)
11701170
Datum eqresult;
11711171

11721172
fcinfo->isnull = false;
1173-
eqresult = (op->d.func.fn_addr) (fcinfo);
1173+
eqresult = op->d.func.fn_addr(fcinfo);
11741174
/* Must invert result of "="; safe to do even if null */
11751175
*op->resvalue = BoolGetDatum(!DatumGetBool(eqresult));
11761176
*op->resnull = fcinfo->isnull;
@@ -1192,7 +1192,7 @@ ExecInterpExpr(ExprState *state, ExprContext *econtext, bool *isnull)
11921192
Datum result;
11931193

11941194
fcinfo->isnull = false;
1195-
result = (op->d.func.fn_addr) (fcinfo);
1195+
result = op->d.func.fn_addr(fcinfo);
11961196

11971197
/* if the arguments are equal return null */
11981198
if (!fcinfo->isnull && DatumGetBool(result))
@@ -1279,7 +1279,7 @@ ExecInterpExpr(ExprState *state, ExprContext *econtext, bool *isnull)
12791279

12801280
/* Apply comparison function */
12811281
fcinfo->isnull = false;
1282-
*op->resvalue = (op->d.rowcompare_step.fn_addr) (fcinfo);
1282+
*op->resvalue = op->d.rowcompare_step.fn_addr(fcinfo);
12831283

12841284
/* force NULL result if NULL function result */
12851285
if (fcinfo->isnull)
@@ -1878,7 +1878,7 @@ ExecEvalParamExtern(ExprState *state, ExprEvalStep *op, ExprContext *econtext)
18781878

18791879
/* give hook a chance in case parameter is dynamic */
18801880
if (!OidIsValid(prm->ptype) && paramInfo->paramFetch != NULL)
1881-
(*paramInfo->paramFetch) (paramInfo, paramId);
1881+
paramInfo->paramFetch(paramInfo, paramId);
18821882

18831883
if (likely(OidIsValid(prm->ptype)))
18841884
{
@@ -3000,7 +3000,7 @@ ExecEvalScalarArrayOp(ExprState *state, ExprEvalStep *op)
30003000
else
30013001
{
30023002
fcinfo->isnull = false;
3003-
thisresult = (op->d.scalararrayop.fn_addr) (fcinfo);
3003+
thisresult = op->d.scalararrayop.fn_addr(fcinfo);
30043004
}
30053005

30063006
/* Combine results per OR or AND semantics */

‎src/backend/executor/execMain.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -349,7 +349,7 @@ standard_ExecutorRun(QueryDesc *queryDesc,
349349
queryDesc->plannedstmt->hasReturning);
350350

351351
if (sendTuples)
352-
(*dest->rStartup) (dest, operation, queryDesc->tupDesc);
352+
dest->rStartup(dest, operation, queryDesc->tupDesc);
353353

354354
/*
355355
* run plan
@@ -375,7 +375,7 @@ standard_ExecutorRun(QueryDesc *queryDesc,
375375
* shutdown tuple receiver, if we started it
376376
*/
377377
if (sendTuples)
378-
(*dest->rShutdown) (dest);
378+
dest->rShutdown(dest);
379379

380380
if (queryDesc->totaltime)
381381
InstrStopNode(queryDesc->totaltime, estate->es_processed);
@@ -1752,7 +1752,7 @@ ExecutePlan(EState *estate,
17521752
* has closed and no more tuples can be sent. If that's the case,
17531753
* end the loop.
17541754
*/
1755-
if (!((*dest->receiveSlot) (slot, dest)))
1755+
if (!dest->receiveSlot(slot, dest))
17561756
break;
17571757
}
17581758

‎src/backend/executor/execParallel.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1081,5 +1081,5 @@ ParallelQueryMain(dsm_segment *seg, shm_toc *toc)
10811081
/* Cleanup. */
10821082
dsa_detach(area);
10831083
FreeQueryDesc(queryDesc);
1084-
(*receiver->rDestroy) (receiver);
1084+
receiver->rDestroy(receiver);
10851085
}

0 commit comments

Comments
 (0)