obsolete reference to a SubPlan field - Mailing list pgsql-hackers

From Amit Langote
Subject obsolete reference to a SubPlan field
Date
Msg-id CA+HiwqHaOx5+V7x7V3H8sWn=Mz0j4q4ZAkM+YtAikTzczkSe6w@mail.gmail.com
Whole thread Raw
Responses Re: obsolete reference to a SubPlan field  (Robert Haas <robertmhaas@gmail.com>)
List pgsql-hackers
I noticed $subject while looking at something involving SubLinks and
SubPlans.  It seems eab6b8b27eb removed the "plan" field from the
SubPlan node struct definition, but the following line from
expression_tree_mutator():

                /* but not the sub-Plan itself, which is referenced as-is */

and the following from expression_tree_walker():

               /* recurse into the testexpr, but not into the Plan */

both of which I think refer to that no-longer-existent field, appear
to have survived multiple commits that moved the SubPlan expression
processing code around.

Attached patch removes those.

-- 
Amit Langote
EDB: http://www.enterprisedb.com

Attachment

pgsql-hackers by date:

Previous
From: Kyotaro Horiguchi
Date:
Subject: Re: RFC: Logging plan of the running query
Next
From: Kyotaro Horiguchi
Date:
Subject: Re: Plug minor memleak in pg_dump