Postgres Pro
Downloads
Home   >   mailing lists

Re: Avoid unused value (src/fe_utils/print.c) - Mailing list pgsql-hackers

From Karina Litskevich
Subject Re: Avoid unused value (src/fe_utils/print.c)
Date
Msg-id CACiT8ib5SY_He-QCwMNg5rGZ4N46sEV_TKk_Qmjg50sDQWGXCA@mail.gmail.com
Whole thread Raw
In response to Re: Avoid unused value (src/fe_utils/print.c)  (Ranier Vilela <ranier.vf@gmail.com>)
Responses Re: Avoid unused value (src/fe_utils/print.c)
List pgsql-hackers
Tree view


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?

Best regards,
Karina Litskevich
Postgres Professional: http://postgrespro.com/
Attachment

pgsql-hackers by date:

Previous
From: Tomas Vondra
Date:
Subject: Re: Use of additional index columns in rows filtering
Next
From: Etsuro Fujita
Date:
Subject: Re: postgres_fdw: wrong results with self join + enable_nestloop off