On Wed, Aug 21, 2024 at 09:29:04AM +0200, Anthonin Bonnefoy wrote:
> Here's the patch with \bindx renamed to \bind_named.

Looks OK to me.  I have spent more time double-checking the whole, and
it looks like we're there, so applied.  Now let's play with it in more
regression tests.  Note that the refactoring patch has been merged
with the original one in a single commit.

> I've also made a small change to Michael's refactoring in 0002 by
> initialising success to false in ExecQueryAndProcessResults. There was
> a compiler warning about success possibly used uninitialized[1].
> 
> [1] https://cirrus-ci.com/task/6207675187331072

Ah, thanks!  I've missed this one.  I see where my mistake was.
--
Michael

Attachment: signature.asc
Description: PGP signature

Reply via email to