Re: Replacing lfirst() with lfirst_node() appropriately in planner.c
Ashutosh Bapat <ashutosh.bapat@enterprisedb.com>
From: Ashutosh Bapat <ashutosh.bapat@enterprisedb.com>
To: Alvaro Herrera <alvherre@2ndquadrant.com>
Cc: pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2017-07-14T08:55:11Z
Lists: pgsql-hackers
On Thu, Jul 13, 2017 at 9:15 PM, Alvaro Herrera <alvherre@2ndquadrant.com> wrote: > Ashutosh Bapat wrote: > >> Happened to stumble across some instances of lfirst() which could use >> lfirst_node() in planner.c. Here's patch which replaces calls to >> lfirst() extracting node pointers by lfirst_node() in planner.c. > > Sounds good. > >> Are we carrying out such replacements in master or this will be >> considered for v11? > > This is for pg11 definitely; please add to the open commitfest. Thanks. Added. https://commitfest.postgresql.org/14/1195/ -- Best Wishes, Ashutosh Bapat EnterpriseDB Corporation The Postgres Database Company
Commits
-
Use lfirst_node() and linitial_node() where appropriate in planner.c.
- 6e427aa4e5f3 11.0 landed