Re: Possible uninitialized use of the variables (src/backend/access/transam/twophase.c)

Поиск
Список
Период
Сортировка
От Michael Paquier
Тема Re: Possible uninitialized use of the variables (src/backend/access/transam/twophase.c)
Дата
Msg-id Ygm5V91SwQPenkFn@paquier.xyz
обсуждение исходный текст
Ответ на Re: Possible uninitialized use of the variables (src/backend/access/transam/twophase.c)  (Michael Paquier <michael@paquier.xyz>)
Ответы Re: Possible uninitialized use of the variables (src/backend/access/transam/twophase.c)  (Kyotaro Horiguchi <horikyota.ntt@gmail.com>)
Re: Possible uninitialized use of the variables (src/backend/access/transam/twophase.c)  (Ranier Vilela <ranier.vf@gmail.com>)
Список pgsql-hackers
Дерево обсуждения 
On Thu, Feb 10, 2022 at 11:38:44AM +0900, Michael Paquier wrote:
> The proposed change is incomplete anyway once you consider this
> argument.  First, there is no need to set up those fields in
> EndPrepare() anymore if there is no origin data, no?  It would be good
> to comment that these are filled in EndPrepare(), I guess, once you do
> the initialization in StartPrepare().

That still looked better on a fresh look in terms of consistency, so
applied this way.
--
Michael
Вложения

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

Предыдущее
От: Michael Paquier
Дата:
Сообщение: Re: pgsql: Add TAP test to automate the equivalent of check_guc
Следующее
От: Thomas Munro
Дата:
Сообщение: Re: buildfarm warnings