On 04.10.22 06:18, Michael Paquier wrote:
> On Mon, Oct 03, 2022 at 08:05:57AM -0300, Ranier Vilela wrote:
>> Em seg., 3 de out. de 2022 às 05:01, Masahiko Sawada <sawada.mshk@gmail.com>
>> escreveu:
>>> On Fri, Sep 30, 2022 at 9:08 AM Ranier Vilela <ranier.vf@gmail.com> wrote:
>>>> 1. Avoid useless reassigning var _logsegno
>>> (src/backend/access/transam/xlog.c)
>>>> Commit 7d70809 left a little oversight.
>>>> XLByteToPrevSeg and XLByteToSeg are macros, and both assign _logsegno.
>>>> So, the first assignment is lost and is useless.
>
> Right, I have missed this one. We do that now in
> build_backup_content() when building the contents of the backup
> history file.
Is this something you want to follow up on, since you were involved in
that patch? Is the redundant assignment simply to be deleted, or do you
want to check the original patch again for context?