Re: cleanup in code

Поиск
Список
Период
Сортировка
От David Rowley
Тема Re: cleanup in code
Дата
Msg-id CAApHDvq6z24xSPxX5ouaC-zR3OXriVxEdOyWL9bwUiNM70RRcA@mail.gmail.com
обсуждение исходный текст
Ответ на Re: cleanup in code  (Heikki Linnakangas <hlinnakangas@vmware.com>)
Ответы Re: cleanup in code  (Andres Freund <andres@2ndquadrant.com>)
Список pgsql-hackers
On Mon, Jan 6, 2014 at 11:38 PM, Heikki Linnakangas <hlinnakangas@vmware.com> wrote:
On 01/04/2014 07:20 AM, Amit Kapila wrote:
1. compiling with msvc shows warning in relcache.c
1>e:\workspace\postgresql\master\postgresql\src\backend\utils\cache\relcache.c(3959):
warning C4715: 'RelationGetIndexAttrBitmap' : not all control paths
return a value

Attached patch remove_msvc_warning.patch to remove above warning

Hmm, I thought we gave enough hints in the elog macro to tell the compiler that elog(ERROR) does no return, since commit b853eb97182079dcd30b4f52576bd5d6c275ee71. Have we not enabled that for MSVC?


I looked at this a while back here:

And found that because elevel was being assigned to a variable that the compiler could not determine that the if (elevel_ >= ERROR) was constant therefore couldn't assume that __assume(0) would be reached with the microsoft compiler

Regards

David Rowley

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

Предыдущее
От: Heikki Linnakangas
Дата:
Сообщение: Re: cleanup in code
Следующее
От: Sandeep Thakkar
Дата:
Сообщение: Re: Compiling extensions on Windows