Re: tracking memory leaks and suggestion

From: Peter H. L. Christiansen (pchristi@nbi.dk)
Date: Wed Jun 14 2000 - 13:24:08 EDT

  • Next message: Konstantin Olchanski: "tpcped.exe changed"

    Hi Flemming, Kris, Anders and all you other g..(.)s
    
    It is clear to me that I don't fully understand the concept of ownership,
    I will have to do a little study.
    
    It was very helpful for me to get Kris comment, but (propably because of
    ownership;) I can only see the objects in my program ( Where I found a
    mess ) and not the objects created inside other objects, so I still think
    it would be nice to have these static variables for debugging. 
    ( It is a big experience to watch 40,000 BrVirtualTrack vanish ;)
    
    I found the leak in the BrTrackFollowFinder which I myself had introduced
    and will remove it soon.
      fTrackGroups->Clear(); should be   fTrackGroups->Delete();
    
    Now it seems that there are no more leaks at least I can do a lot of
    tracking ~ 100 * multiplicity 2-60 ~ 180.000 tracks in TPM1 without
    any sign of ruining my memory. (digitization, clustering, and tracking).
     
    > I guess you don'nt need the
    > >    fClusterTable = 0;
    > since thenobject is deleted anyhow.
    
    I started doing this since I read in my c++ manual that it was nice to do,
    so you don't have the pointer pointing to junk, but it might be, that it
    is done automatically. I don't know.
    > 
    > You have to delete the seq. By adding the address cluster will come in two
    > flavours
    > - the single clusters that have the input sequences. (owned by event_node)
    > and
    > - deconvoluted new sequences  that thus would be owned by the cluster.
    > 
    > To make the ownership clear a copy was made.
    
    OK
    > 
    > 
    > -- Do you have the most recent changes by Trine who worked on these methods?
    > 
    Propably not, I will update soon since it seems that I no longer need my
    messy brat version.
    
    Thanks for the useful comments
       Peter
    



    This archive was generated by hypermail 2b29 : Wed Jun 14 2000 - 13:28:42 EDT