Horton

Thread

Commits

  1. Constify proclist.h

  1. [PATCH] Constify proclist.h

    Aleksander Alekseev <aleksander@timescale.com> — 2023-01-17T12:18:06Z

    Hi hackers,

This is a follow-up to [1] and c8ad4d81.

> Additionally Bharath pointed out that there are other pieces of code
> that we may want to change in a similar fashion,
> proclist.h/proclist_types.h as one example. I didn't do this yet
> because I would like to know the community opinion first on whether we
> should do this at all.

Since the consensus seems to be to constify everything possible here
is the patch for proclist.h. There is nothing to change in
proclist_types.h.

[1]: https://postgr.es/m/CAJ7c6TM2%3D08mNKD9aJg8vEY9hd%2BG4L7%2BNvh30UiNT3kShgRgNg%40mail.gmail.com

-- 
Best regards,
Aleksander Alekseev

  2. Re: [PATCH] Constify proclist.h

    Peter Eisentraut <peter.eisentraut@enterprisedb.com> — 2023-01-19T08:52:57Z

    On 17.01.23 13:18, Aleksander Alekseev wrote:
> This is a follow-up to [1] and c8ad4d81.
> 
>> Additionally Bharath pointed out that there are other pieces of code
>> that we may want to change in a similar fashion,
>> proclist.h/proclist_types.h as one example. I didn't do this yet
>> because I would like to know the community opinion first on whether we
>> should do this at all.
> 
> Since the consensus seems to be to constify everything possible here
> is the patch for proclist.h. There is nothing to change in
> proclist_types.h.
> 
> [1]: https://postgr.es/m/CAJ7c6TM2%3D08mNKD9aJg8vEY9hd%2BG4L7%2BNvh30UiNT3kShgRgNg%40mail.gmail.com

committed