Pavlin Radoslavov wrote:
> Ben Greear <[EMAIL PROTECTED]> wrote:
> 
>> It seems that the TimerList object (timer.cc)
>> doesn't delete the members in it's _heaplist
>> in the destructor.
>>
>> Are these supposed to be cleaned up somewhere else?
> 
> I think you are right, it seems the Heap objects on the _heaplist
> are not deleted when TimerList destructor is invoked.
> The Heap objects are alocated only inside TimerList::find_heap() and
> the _heaplist management/access is internal to TimerList only, so
> the TimerList destructor should delete the leftover.
> 
> The fix for that will be trivial, but lets address it after the
> pending two xorpsh related patches are out of the way.

I thought it would be trivial too, but ran into wierd problems.  Hopefully
it's some other mistake I added in my code...will re-sync with your tree
when you commit the changes.

Thanks,
Ben

> 
> Thanks,
> Pavlin


-- 
Ben Greear <[EMAIL PROTECTED]>
Candela Technologies Inc  http://www.candelatech.com

_______________________________________________
Xorp-hackers mailing list
[email protected]
http://mailman.ICSI.Berkeley.EDU/mailman/listinfo/xorp-hackers

Reply via email to