Avoid unncessary always true test (src/backend/storage/buffer/bufmgr.c)

Поиск
Список
Период
Сортировка
От Ranier Vilela
Тема Avoid unncessary always true test (src/backend/storage/buffer/bufmgr.c)
Дата
Msg-id CAEudQAoSA7saBF7RaoYbRjE+Zojojin0Qc9A4hebPKB8o1J+EQ@mail.gmail.com
обсуждение исходный текст
Ответы Re: Avoid unncessary always true test (src/backend/storage/buffer/bufmgr.c)  (Richard Guo <guofenglinux@gmail.com>)
Список pgsql-hackers
Hi,

Per Coverity.

At function ExtendBufferedRelShared, has a always true test.
eb.rel was dereferenced one line above, so in
if (eb.rel) is always true.

I think it's worth removing the test, because Coverity raises dozens of alerts thinking eb.rel might be NULL.
Besides, one less test is one less branch.

regards,
Ranier Vilela
Вложения

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

Предыдущее
От: Ranier Vilela
Дата:
Сообщение: Re: Avoid unused value (src/fe_utils/print.c)
Следующее
От: Amit Kapila
Дата:
Сообщение: Re: Wrong syntax in feature description