Um.  That's the desired behaviour, else there's going to be a mem leak.  the
'meta' in this instance is a local variable -- why set it to null when the
function returns a line later?  If you're having problems with this, I'd say
it's because you changed the MetaData class so that it needs an explicit copy
constructor...

Isaac 

On 27-Sep-2000 Chris Kuklewicz wrote:
> First: WARNING the function 
> 
> Error MusicCatalog::AddSong(const char *url)
> 
> says "delete meta;" inside and after the look... twice...segfault
> 
> STRONG SUGGESTION : after a call to delete set the pointer to NULL.
> Only exception is if it is manifestly obvious from looking at the
> screen that it cannot possibly screw up.  (like a two line function).
> 
> Setting meta=NULL after a delete prevents this problem here.
> 
> I am hacking other things too much to sumbit a patch yet.
_______________________________________________
[EMAIL PROTECTED]
http://www.freeamp.org/mailman/listinfo/freeamp-dev

Reply via email to