On Mon, Jul 20, 2015 at 06:00:53PM +0100, Tim Bunce wrote: > On Mon, Jul 20, 2015 at 02:54:53PM +0100, Martin J. Evans wrote: > > I also noticed something I should have seen before: dbd_st_rows() is > defined as returning an int. I _think_ it would be safe to change the > definition to returning an IV since it's only used internally by drivers > via the Driver.xst template file that does: > > XST_mIV(0, dbd_st_rows(sth, imp_sth));
Ah. The same logic also means I could change the return type of dbd_st_execute and dbd_db_do4, which also would help. Tim.