On Tue, Jun 06, 2006 at 06:22:49 -0700, Evan Nemerson wrote:
> I might be reading it wrong, but I think erase_data() (in import.c) will
> always return 0. The return value is initialised to 0, incremented by
> the return value of itself (called recursively), and returned. Those are
> the only references to it.
> Glancing through the related functions, it seems like this would
> decrease the likelihood of ts_orphan_cleanup() being called.
ts_orphan_cleanup() wont be called anyway, it only for the non-quad store,
which is deprecated. The nono-quad code should really just be removed.
> Anyways, I think adding ret += mysql_affected_rows(db->db); after the
> DELETE query would implement be the desired behaviour, but I'm by no
> means an expert so I don't want to commit it.
Yep, that's right.