karlseguin / pg.zig

Native PostgreSQL driver / client for Zig
MIT License
158 stars 9 forks source link

prepare does a start flow - it should endFlow on the way out #20

Closed zigster64 closed 1 month ago

zigster64 commented 1 month ago

applying just this change on top of your arena fix removed the bus error

To test this, I added a tonne of debug.print statements to show the values of the reader.allocator and reader.default_allocator addresses.

It's all good until the first statement gets prepared, and then the reader.allocator gets out of synch. This patch fixes that, and the 2 values stay correctly in synch from there on

zigster64 commented 1 month ago

No, that’s incorrect sorry

the prepare starts the flow, the stmt ends the flow after execute

ignore this pr Something else is causing it