Re: [jazzplusplus-devel] Core dumping

[Dave F. <dav...@co...>]

On Friday 26 March 2004 05:38 am, Matt Kelly wrote:
> In events.h:
>
> Line 546-ish:
>
>      virtual ~tMetaEvent()
>      {
>        delete [] Data;
>      }
>
> In track.cpp:
>
> diff -r1.4 track.cpp
> 1111,1116c1111,1116
> <   tJazzMeta jazz;
> <   jazz.SetAudioMode(audio_mode);
> <   jazz.SetTrackState(State);
> <   jazz.SetTrackDevice(Device);
> <   jazz.SetIntroLength(TheSong->GetIntroLength());
> <   jazz.Write(io);
> ---
>
>  >   tJazzMeta *jazz = new tJazzMeta;
>  >   jazz->SetAudioMode(audio_mode);
>  >   jazz->SetTrackState(State);
>  >   jazz->SetTrackDevice(Device);
>  >   jazz->SetIntroLength(TheSong->GetIntroLength());
>  >   jazz->Write(io);
>
> In jazz.cpp -- the first one is in function
> tConfigEntry::~tConfigEntry(), and the second one is in
> tConfig::~tConfig():
>
>
> diff -r1.6 jazz.cpp
> 101c101,102
> <   delete StrValue;
> ---
>
>  >   if (StrValue)
>  >      delete [] StrValue;
>
> 274c275,276
> <       delete [] Names[i];
> ---
>
>  >       if (Names[i])
>  >           delete  Names[i];

What type is Names?  wxWidgets is supposed to clean itself up, and if tConfig 
is deleting wxWidgets stuff that the library is cleaning up itself, then 
that's what the problem is.

In any case, it looks like tConfig was written before there was wxConfig, 
because it duplicates all of the function of wxConfig.  Anybody willing to 
take a crack at rewriting tConfig to use wxConfig instead?  It's sorta on the 
list of things to do....  ;)

Dave

> Please apply these to the sources.
>
> Maybe I should have write access to the repository?
>
> Matt
>
>
>
>
>
> -------------------------------------------------------
> This SF.Net email is sponsored by: IBM Linux Tutorials
> Free Linux tutorial presented by Daniel Robbins, President and CEO of
> GenToo technologies. Learn everything from fundamentals to system
> administration.http://ads.osdn.com/?ad_id=1470&alloc_id=3638&op=click
> _______________________________________________
> jazzplusplus-devel mailing list
> jaz...@li...
> https://lists.sourceforge.net/lists/listinfo/jazzplusplus-devel

-- 
Visit my website!
http://www.davefancella.com/?event=em

Never try to outstubborn a cat.
		-- Lazarus Long, "Time Enough for Love"