Re: [HACKERS] assorted code cleanup - Mailing list pgsql-hackers

From Robert Haas
Subject Re: [HACKERS] assorted code cleanup
Date
Msg-id CA+TgmoZaA_wtvAq0V6G+0yViivakjbMyEDPah9Zgi7H9e6iW4Q@mail.gmail.com
Whole thread Raw
In response to Re: [HACKERS] assorted code cleanup  (Michael Paquier <michael.paquier@gmail.com>)
Responses Re: [HACKERS] assorted code cleanup
List pgsql-hackers
On Mon, Aug 21, 2017 at 1:11 AM, Michael Paquier
<michael.paquier@gmail.com> wrote:
> On Fri, Aug 18, 2017 at 1:56 AM, Peter Eisentraut
> <peter.eisentraut@2ndquadrant.com> wrote:
>> Here are a few assorted patches I made while working on the stdbool set,
>> cleaning up various pieces of dead code and weird styles.
>>
>> - Drop excessive dereferencing of function pointers
>
> -           (*next_ProcessUtility_hook) (pstmt, queryString,
> +           next_ProcessUtility_hook(pstmt, queryString,
>                                          context, params, queryEnv,
>                                          dest, completionTag);
> But this... Personally I like the current grammar which allows one to
> make the difference between a function call with something declared
> locally and something that may be going to a custom code path.

+1.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company



pgsql-hackers by date:

Previous
From: Peter Eisentraut
Date:
Subject: Re: [HACKERS] locale problem of bgworker: logical replicationlauncher and worker process
Next
From: Robert Haas
Date:
Subject: Re: [HACKERS] subscription worker signalling wal writer too much