Re: Avoid unused value (src/fe_utils/print.c)

Ranier Vilela <ranier.vf@gmail.com>

From: Ranier Vilela <ranier.vf@gmail.com>

To: Karina Litskevich <litskevichkarina@gmail.com>

Cc: marko@joh.to, Alexander Lakhin <exclusion@gmail.com>, Michael Paquier <michael@paquier.xyz>, Pg Hackers <pgsql-hackers@postgresql.org>

Date: 2023-07-24T22:04:27Z

Lists: pgsql-hackers

Em sex., 21 de jul. de 2023 às 09:13, Karina Litskevich <
litskevichkarina@gmail.com> escreveu:

>
>
> On Wed, Jul 12, 2023 at 1:46 AM Ranier Vilela <ranier.vf@gmail.com> wrote:
>
>>  As there is consensus to keep the no-op assignment,
>> I will go ahead and reject the patch.
>>
>
> In your patch you suggest removing two assignments, and we only have
> consensus to keep one of them. The other one is an obvious no-op.
>
> I attached a patch that removes only one assignment. Could you please try
> it and check whether Coverity is still complaining about need_recordsep
> variable?
>
Yeah.
Checked today, Coverity does not complain about need_recordsep.

best regards,
Ranier Vilela