From 2df807d5205f1852b35b565b58e6a2f854cff37c Mon Sep 17 00:00:00 2001 From: Henson Choi Date: Tue, 16 Jun 2026 23:26:16 +0900 Subject: [PATCH] Remove blank-line changes unrelated to row pattern recognition The RPR patch added or dropped blank lines in five existing files with no related code change. Revert them so the files differ from the base only where RPR actually touches them. --- src/backend/executor/nodeWindowAgg.c | 3 --- src/backend/nodes/equalfuncs.c | 1 - src/backend/optimizer/plan/setrefs.c | 1 + src/backend/parser/parse_clause.c | 1 + src/backend/utils/adt/ruleutils.c | 1 - src/backend/utils/adt/windowfuncs.c | 1 + 6 files changed, 3 insertions(+), 5 deletions(-) diff --git a/src/backend/executor/nodeWindowAgg.c b/src/backend/executor/nodeWindowAgg.c index cb6a484b7de..778cebff934 100644 --- a/src/backend/executor/nodeWindowAgg.c +++ b/src/backend/executor/nodeWindowAgg.c @@ -177,7 +177,6 @@ typedef struct WindowStatePerAggData bool restart; /* need to restart this agg in this cycle? */ } WindowStatePerAggData; - static void initialize_windowaggregate(WindowAggState *winstate, WindowStatePerFunc perfuncstate, WindowStatePerAgg peraggstate); @@ -1024,7 +1023,6 @@ eval_windowaggregates(WindowAggState *winstate) agg_row_slot, false); if (ret < 0) break; - if (ret == 0) goto next_tuple; @@ -1086,7 +1084,6 @@ next_tuple: ExecClearTuple(agg_row_slot); } - /* The frame's end is not supposed to move backwards, ever */ Assert(aggregatedupto_nonrestarted <= winstate->aggregatedupto); diff --git a/src/backend/nodes/equalfuncs.c b/src/backend/nodes/equalfuncs.c index 328199918b8..d3fbf21bf2e 100644 --- a/src/backend/nodes/equalfuncs.c +++ b/src/backend/nodes/equalfuncs.c @@ -20,7 +20,6 @@ #include "postgres.h" #include "miscadmin.h" -#include "nodes/plannodes.h" #include "utils/datum.h" diff --git a/src/backend/optimizer/plan/setrefs.c b/src/backend/optimizer/plan/setrefs.c index bb8cdc97c76..46229312d8a 100644 --- a/src/backend/optimizer/plan/setrefs.c +++ b/src/backend/optimizer/plan/setrefs.c @@ -211,6 +211,7 @@ static List *set_windowagg_runcondition_references(PlannerInfo *root, static void record_elided_node(PlannerGlobal *glob, int plan_node_id, NodeTag elided_type, Bitmapset *relids); + /***************************************************************************** * * SUBPLAN REFERENCES diff --git a/src/backend/parser/parse_clause.c b/src/backend/parser/parse_clause.c index 3d22bc8432c..3513c57823e 100644 --- a/src/backend/parser/parse_clause.c +++ b/src/backend/parser/parse_clause.c @@ -102,6 +102,7 @@ static Node *transformFrameOffset(ParseState *pstate, int frameOptions, Oid rangeopfamily, Oid rangeopcintype, Oid *inRangeFunc, Node *clause); + /* * transformFromClause - * Process the FROM clause and add items to the query's range table, diff --git a/src/backend/utils/adt/ruleutils.c b/src/backend/utils/adt/ruleutils.c index b5686e010c0..2930275c793 100644 --- a/src/backend/utils/adt/ruleutils.c +++ b/src/backend/utils/adt/ruleutils.c @@ -7313,7 +7313,6 @@ get_rule_windowspec(WindowClause *wc, List *targetList, get_rule_orderby(wc->orderClause, targetList, false, context); needspace = true; } - /* framing clause is never inherited, so print unless it's default */ if (wc->frameOptions & FRAMEOPTION_NONDEFAULT) { diff --git a/src/backend/utils/adt/windowfuncs.c b/src/backend/utils/adt/windowfuncs.c index 3869f6c8994..d15aa0c75db 100644 --- a/src/backend/utils/adt/windowfuncs.c +++ b/src/backend/utils/adt/windowfuncs.c @@ -41,6 +41,7 @@ static bool rank_up(WindowObject winobj); static Datum leadlag_common(FunctionCallInfo fcinfo, bool forward, bool withoffset, bool withdefault); + /* * utility routine for *_rank functions. */