Crossfire Mailing List Archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: CF: Re: apply() cleanup



On Wed, May 10, 2000 at 09:57:14PM -0700, Mark Wedel wrote:
>  Your right.  But it wouldn't be hard to change the free_object function to
> clear the count value to something..

I've done this and uncovered some problems.  When removing all objects
from a map with free_all_objects(), a button is removed, then something
else is removed that is on top of another button, that other button is
updated via update_button(), and update_button() complains loudly
because the first button is still in the get_button_links() list, but
already freed.  This wasn't triggered before because update_button()
only checks op->count but not QUERY_FLAG (op, FLAG_FREED).

I'm also wondering how move_all_objects() is supposed to work because
it doesn't update the button links.

-- 
Jan
-
[you can put yourself on the announcement list only or unsubscribe altogether
by sending an email stating your wishes to crossfire-request@ifi.uio.no]