Re: [plpython] Add missing volatile qualifier. - Mailing list pgsql-hackers

From Tom Lane
Subject Re: [plpython] Add missing volatile qualifier.
Date
Msg-id 1145662.1711996745@sss.pgh.pa.us
Whole thread Raw
In response to Re: [plpython] Add missing volatile qualifier.  (Nathan Bossart <nathandbossart@gmail.com>)
List 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 by date:

Previous
From: Nathan Bossart
Date:
Subject: Re: [plpython] Add missing volatile qualifier.
Next
From: Corey Huinker
Date:
Subject: Re: Statistics Import and Export