Re: Convert *GetDatum() and DatumGet*() macros to inline functions - Mailing list pgsql-hackers

From Peter Eisentraut
Subject Re: Convert *GetDatum() and DatumGet*() macros to inline functions
Date
Msg-id 61c44649-2a70-1a83-48d3-aa79203686f8@enterprisedb.com
Whole thread Raw
In response to Re: Convert *GetDatum() and DatumGet*() macros to inline functions  (Aleksander Alekseev <aleksander@timescale.com>)
Responses Re: Convert *GetDatum() and DatumGet*() macros to inline functions
List pgsql-hackers
On 08.09.22 11:26, Aleksander Alekseev wrote:
>>> 3. Go with your patch and just fix up the warnings about uninitialized
>>> variables.  But that seems the least principled to me.
>>
>> IMO the 3rd option is the lesser evil. Initializing four bools/ints in
>> order to make Clang 11 happy doesn't strike me as such a big deal. At
>> least until somebody reports a bottleneck for this particular reason.
>> We can optimize the code when and if this will happen.
> 
> Since the first patch was applied, cfbot now complains that it can't
> apply the patchset. Here is the rebased version.

committed, thanks




pgsql-hackers by date:

Previous
From: Peter Eisentraut
Date:
Subject: Re: preserve timestamps when installing headers
Next
From: Julien Rouhaud
Date:
Subject: Re: Support for Rust