Re: [HACKERS] Replacing lfirst() with lfirst_node() appropriately inplanner.c - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Re: [HACKERS] Replacing lfirst() with lfirst_node() appropriately inplanner.c
Date
Msg-id 20170713154501.hmiix4sbt7c24reo@alvherre.pgsql
Whole thread Raw
In response to [HACKERS] Replacing lfirst() with lfirst_node() appropriately in planner.c  (Ashutosh Bapat <ashutosh.bapat@enterprisedb.com>)
Responses Re: [HACKERS] Replacing lfirst() with lfirst_node() appropriately in planner.c
List pgsql-hackers
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.

-- 
Álvaro Herrera                https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services



pgsql-hackers by date:

Previous
From: Heikki Linnakangas
Date:
Subject: Re: [HACKERS] PostgreSQL - Weak DH group
Next
From: David Kohn
Date:
Subject: Re: [HACKERS] Function Volatility and Views Unexpected Behavior