My replying skills are subpar! :) Anyways, this is the only chunk that
really sticks out at me. I don't know Runters code to well. But I
think you might want to make 'to' a pointer. Looks I'm pretty sure
each mask is a linked list of bits (32 bits a node) so just doing a
'to = mask' (asuming it's just straight C and no operator overloading
is going on) will not suffice. Instead, make 'to' a pointer, and have
it modified. mask might be going out of scope (and once again,
assuming there's no destructor for this) the memory will just linger
on. Ya... sorry about the blank reply too!
Davion
> void copy_bitmask(BITMASK to, BITMASK *from)
> {
> BITMASK mask;
> BMlist *pBlist;
>
> for(pBlist = to.int_list;pBlist;pBlist = pBlist->next)//remove all the
bits
> remove_bit(&to,pBlist->tar_mask);
>
> init_bitmask(&mask);
> for(pBlist = from->int_list;pBlist;pBlist = pBlist->next)//add the new
ones
> set_bit(&mask,pBlist->tar_mask);
> to = mask;
> return;
>
> }
--
ROM mailing list
[email protected]
Unsubscribe here ->>> http://www.rom.org/cgi-bin/mailman/listinfo/rom
