On Tue, Jul 11, 2023 at 1:43 PM Gurjeet Singh <gurj...@singh.im> wrote: > > In the above hunk, if there's an exception/ERROR, I believe we should > PG_RE_THROW(). If there's a reason to continue, we should at least set > rslot = NULL, otherwise we may be returning an uninitialized value to > the caller.
Excuse the brain-fart on my part. There's not need to PG_RE_THROW(), since there's no PG_CATCH(). Re-learning the code's infrastructure slowly :-) Best regards, Gurjeet http://Gurje.et