From 8126eaee2fed7cbc4e9fd6ba8713977ccacd77fe Mon Sep 17 00:00:00 2001 From: Robert Haas Date: Wed, 27 Apr 2016 11:29:45 -0400 Subject: Clean up a few parallelism-related things that pgindent wants to mangle. In nodeFuncs.c, pgindent wants to introduce spurious indentation into the definitions of planstate_tree_walker and planstate_walk_subplans. Fix that by spreading the definition out across several lines, similar to what is already done for other walker functions in that file. In execParallel.c, in the definition of SharedExecutorInstrumentation, pgindent wants to insert more whitespace between the type name and the member name. That causes it to mangle comments later on the line. Fix by moving the comments out of line. Now that we have a bit more room, add some more details that may be useful to the next person reading this code. --- src/backend/nodes/nodeFuncs.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) (limited to 'src/backend/nodes/nodeFuncs.c') diff --git a/src/backend/nodes/nodeFuncs.c b/src/backend/nodes/nodeFuncs.c index 46af8721161..7bc5be1565f 100644 --- a/src/backend/nodes/nodeFuncs.c +++ b/src/backend/nodes/nodeFuncs.c @@ -3425,7 +3425,9 @@ raw_expression_tree_walker(Node *node, * recurse into any sub-nodes it has. */ bool -planstate_tree_walker(PlanState *planstate, bool (*walker) (), void *context) +planstate_tree_walker(PlanState *planstate, + bool (*walker) (), + void *context) { Plan *plan = planstate->plan; ListCell *lc; @@ -3507,7 +3509,9 @@ planstate_tree_walker(PlanState *planstate, bool (*walker) (), void *context) * Walk a list of SubPlans (or initPlans, which also use SubPlan nodes). */ static bool -planstate_walk_subplans(List *plans, bool (*walker) (), void *context) +planstate_walk_subplans(List *plans, + bool (*walker) (), + void *context) { ListCell *lc; -- cgit v1.2.3