Re: plenty code is confused about function level static - Mailing list pgsql-hackers

From Peter Eisentraut
Subject Re: plenty code is confused about function level static
Date
Msg-id e494aa10-a6ce-4d0f-890d-86ae3e4f8530@eisentraut.org
Whole thread Raw
In response to plenty code is confused about function level static  (Andres Freund <andres@anarazel.de>)
Responses Re: plenty code is confused about function level static
List pgsql-hackers
On 17.04.24 23:39, Andres Freund wrote:
> Is there some reason we went for this pattern in a fair number of places? I
> assume it's mostly copy-pasta, but...

Right.  I don't think it is commonly understood that adding const 
qualifiers can help compiler optimization, and it's difficult to 
systematically check for omissions or verify the optimization effects. 
So I think we just have to keep trying to do our best manually for now.

> Attached are fixes for struct option and a few more occurrences I've found
> with a bit of grepping.

These look good to me.




pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: fix tablespace handling in pg_combinebackup
Next
From: "Andrey M. Borodin"
Date:
Subject: Re: plenty code is confused about function level static