[PATCH] Constify proclist.h

Started by Aleksander Alekseevabout 3 years ago2 messageshackers
Jump to latest
#1Aleksander Alekseev
aleksander@timescale.com

Hi hackers,

This is a follow-up to [1]/messages/by-id/CAJ7c6TM2=08mNKD9aJg8vEY9hd+G4L7+Nvh30UiNT3kShgRgNg@mail.gmail.com 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]: /messages/by-id/CAJ7c6TM2=08mNKD9aJg8vEY9hd+G4L7+Nvh30UiNT3kShgRgNg@mail.gmail.com

--
Best regards,
Aleksander Alekseev

Attachments:

v1-0001-Constify-proclist.h.patchapplication/octet-stream; name=v1-0001-Constify-proclist.h.patchDownload+3-4
#2Peter Eisentraut
peter_e@gmx.net
In reply to: Aleksander Alekseev (#1)
Re: [PATCH] Constify proclist.h

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]: /messages/by-id/CAJ7c6TM2=08mNKD9aJg8vEY9hd+G4L7+Nvh30UiNT3kShgRgNg@mail.gmail.com

committed