Re: [plpython] Add missing volatile qualifier.

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: [plpython] Add missing volatile qualifier.
Дата
Msg-id 1145662.1711996745@sss.pgh.pa.us
обсуждение исходный текст
Ответ на Re: [plpython] Add missing volatile qualifier.  (Nathan Bossart <nathandbossart@gmail.com>)
Список pgsql-hackers
Nathan Bossart <nathandbossart@gmail.com> writes:
> On Mon, Apr 01, 2024 at 11:57:07AM -0400, Tom Lane wrote:
>> Good catch!  It looks like the consequences of a failure would be
>> pretty minimal --- AFAICS, no worse than a possible failure to remove
>> a refcount on Py_None --- but that's still a bug.

> Huh.  I seem to have dropped that "volatile" shortly before committing for
> some reason [0].

Oh, I'd forgotten that discussion.  Given that we were both confused
about the need for it, all the more reason to try to avoid using a
within-PG_TRY assignment.

> Your fix seems fine to me.

Thanks for looking, I'll push it shortly.

            regards, tom lane



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

Предыдущее
От: Nathan Bossart
Дата:
Сообщение: Re: [plpython] Add missing volatile qualifier.
Следующее
От: Corey Huinker
Дата:
Сообщение: Re: Statistics Import and Export