Re: Signed vs Unsigned (take 2) (src/backend/storage/ipc/procarray.c)

On 2021/09/11 12:21, Fujii Masao wrote:
>
>
> On 2021/07/23 20:07, Ranier Vilela wrote:
>> Em sex., 23 de jul. de 2021 às 07:02, Aleksander Alekseev <aleksander@timescale.com <mailto:aleksander@timescale.com>> escreveu:
>>
>>     Hi hackers,
>>
>>         The following review has been posted through the commitfest application:
>>         make installcheck-world:  tested, passed
>>         Implements feature:       tested, passed
>>         Spec compliant:           tested, passed
>>         Documentation:            tested, passed
>>
>>         The patch was tested on MacOS against master `80ba4bb3`.
>>
>>         The new status of this patch is: Ready for Committer
>>
>>
>>     The second patch seems fine too. I'm attaching both patches to trigger cfbot and to double-check them.
>>
>> Thanks Aleksander, for reviewing this.
>
> I looked at these patches because they are marked as ready for committer.
> They don't change any actual behavior, but look valid to me in term of coding.
> Barring any objection, I will commit them.

> No need to backpatch, why this patch is classified as
> refactoring only.

I found this in the commit log in the patch. I agree that these patches
are refactoring ones. But I'm thinking that it's worth doing back-patch,
to make future back-patching easy. Thought?