Re: PL/Python: Fix return in the middle of PG_TRY() block.
От
Nathan Bossart
Тема
Re: PL/Python: Fix return in the middle of PG_TRY() block.
Дата
Msg-id
20230504234235.GA2419591@nathanxps13
Ответ на
Список
Дерево обсуждения
PL/Python: Fix return in the middle of PG_TRY() block. Xing Guo <higuoxing@gmail.com>
Re: PL/Python: Fix return in the middle of PG_TRY() block. Nathan Bossart <nathandbossart@gmail.com>
Re: PL/Python: Fix return in the middle of PG_TRY() block. Andres Freund <andres@anarazel.de>
Re: PL/Python: Fix return in the middle of PG_TRY() block. Andres Freund <andres@anarazel.de>
Re: PL/Python: Fix return in the middle of PG_TRY() block. Xing Guo <higuoxing@gmail.com>
Re: PL/Python: Fix return in the middle of PG_TRY() block. Nathan Bossart <nathandbossart@gmail.com>
Re: PL/Python: Fix return in the middle of PG_TRY() block. Xing Guo <higuoxing@gmail.com>
Re: PL/Python: Fix return in the middle of PG_TRY() block. Tom Lane <tgl@sss.pgh.pa.us>
Re: PL/Python: Fix return in the middle of PG_TRY() block. Andres Freund <andres@anarazel.de>
Re: PL/Python: Fix return in the middle of PG_TRY() block. Nathan Bossart <nathandbossart@gmail.com>
Re: PL/Python: Fix return in the middle of PG_TRY() block. Nathan Bossart <nathandbossart@gmail.com>
Re: PL/Python: Fix return in the middle of PG_TRY() block. Tom Lane <tgl@sss.pgh.pa.us>
Re: PL/Python: Fix return in the middle of PG_TRY() block. Nathan Bossart <nathandbossart@gmail.com>
Re: PL/Python: Fix return in the middle of PG_TRY() block. Nathan Bossart <nathandbossart@gmail.com>
Re: PL/Python: Fix return in the middle of PG_TRY() block. Nathan Bossart <nathandbossart@gmail.com>
Re: PL/Python: Fix return in the middle of PG_TRY() block. Tom Lane <tgl@sss.pgh.pa.us>
Re: PL/Python: Fix return in the middle of PG_TRY() block. Nathan Bossart <nathandbossart@gmail.com>
Re: PL/Python: Fix return in the middle of PG_TRY() block. Xing Guo <higuoxing@gmail.com>
Re: PL/Python: Fix return in the middle of PG_TRY() block. Xing Guo <higuoxing@gmail.com>
Re: PL/Python: Fix return in the middle of PG_TRY() block. Michael Paquier <michael@paquier.xyz>
On Thu, May 04, 2023 at 08:39:03AM -0400, Tom Lane wrote: > Hmm, I'm not sure why PLy_trigger_build_args's pltargs needs to > gain a "volatile" here? LGTM otherwise. I removed that new "volatile" marker before committing. I was trying to future-proof a bit and follow elog.h's recommendation to the letter, but following your mental model upthread, it doesn't seem to be strictly necessary, and we'd need to set pltargs to NULL after decrementing its reference count in the PG_TRY section for such future-proofing to be effective, anyway. Thank you for reviewing! -- Nathan Bossart Amazon Web Services: https://aws.amazon.com
В списке pgsql-hackers по дате отправления
От: Andrew Dunstan
Дата: