Re: pgsql: Fix behavior of ~> (cube, int) operator

Поиск
Список
Период
Сортировка
От Alexander Korotkov
Тема Re: pgsql: Fix behavior of ~> (cube, int) operator
Дата
Msg-id CAPpHfdvTakCmjcbAcsaSB0XZ=WCLHRTmA+F2N3P12OCOWZfjBQ@mail.gmail.com
обсуждение исходный текст
Ответ на Re: pgsql: Fix behavior of ~> (cube, int) operator  (Tom Lane <tgl@sss.pgh.pa.us>)
Ответы Re: pgsql: Fix behavior of ~> (cube, int) operator  (Teodor Sigaev <teodor@sigaev.ru>)
Re: pgsql: Fix behavior of ~> (cube, int) operator  (Teodor Sigaev <teodor@sigaev.ru>)
Список pgsql-committers
On Sun, Jan 21, 2018 at 11:20 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
Teodor Sigaev <teodor@sigaev.ru> writes:
> Fix behavior of ~> (cube, int) operator

This patch has caused Coverity to complain, correctly AFAICS, about
dead code in cube_coord_llur in the back branches:

1628            /* Inverse value if needed */
1629            if (inverse)
>>>     CID 1463943:  Control flow issues  (DEADCODE)
>>>     Execution cannot reach this statement: "result = -result;".
1630                    result = -result;
1631
1632            PG_RETURN_FLOAT8(result);

Seems to be due to sloppy division of changes between f50c80dbb (which
was not back-patched) and 563a053bd.  Please fix.

Thank you for catching this.  You diagnosis is right.
I propose to commit the attached patch to 10 and 9.6.

------
Alexander Korotkov
Postgres Professional: http://www.postgrespro.com
The Russian Postgres Company 
Вложения

В списке pgsql-committers по дате отправления:

Предыдущее
От: David Rowley
Дата:
Сообщение: Re: pgsql: Support partition pruning at execution time
Следующее
От: Teodor Sigaev
Дата:
Сообщение: Re: pgsql: Fix behavior of ~> (cube, int) operator