[Date Prev][Date Next]
Re: idl_delete bug in back-ldbm?
> Look closily. If the id was the only one in the list,
> ldbm_cache_delete() is called. If the id isn't the only
> one on the list, the id is removed using SAFEMEMCPY()
> and stores the new list using idl_store().
Ah yes. I didn't realise that a copy operation can also delete.
> >Was this the right thing to do?
> It's probably better to change the caller to ignore the error
> (after logging it). I'll commit such a change to -devel.
True. I haven't needed this patch since regenerating my gdbm database, but
I'm still happy that something good came from this.
Keep up the good work!