Re: Reorganize GUC structs

On 29.10.25 10:07, Peter Eisentraut wrote:
> On 24.10.25 14:21, Heikki Linnakangas wrote:
>>> @@ -261,15 +261,15 @@ static bool
>>> assignable_custom_variable_name(const char *name, bool skip_errors,
>>>                                              int elevel);
>>>  static void do_serialize(char **destptr, Size *maxbytes,
>>>                           const char *fmt,...) pg_attribute_printf(3,
>>> 4);
>>> -static bool call_bool_check_hook(const struct config_bool *conf,
>>> bool *newval,
>>> +static bool call_bool_check_hook(const struct config_generic *conf,
>>> bool *newval,
>>>                                   void **extra, GucSource source, int
>>> elevel);
>>> -static bool call_int_check_hook(const struct config_int *conf, int
>>> *newval,
>>> +static bool call_int_check_hook(const struct config_generic *conf,
>>> int *newval,
>>>                                  void **extra, GucSource source, int
>>> elevel);
>>> -static bool call_real_check_hook(const struct config_real *conf,
>>> double *newval,
>>> +static bool call_real_check_hook(const struct config_generic *conf,
>>> double *newval,
>>>                                   void **extra, GucSource source, int
>>> elevel);
>>> -static bool call_string_check_hook(const struct config_string *conf,
>>> char **newval,
>>> +static bool call_string_check_hook(const struct config_generic
>>> *conf, char **newval,
>>>                                     void **extra, GucSource source,
>>> int elevel);
>>> -static bool call_enum_check_hook(const struct config_enum *conf, int
>>> *newval,
>>> +static bool call_enum_check_hook(const struct config_generic *conf,
>>> int *newval,
>>>                                   void **extra, GucSource source, int
>>> elevel);
>>
>> The new signatures for these function are less specific than before,
>> making them a little worse IMO. Overall +1 on the patches, despite
>> that little drawback.
>
> Thanks, pushed.

The remaining patches to sort the list alphabetically have also been pushed.