[SDL] SDL Mixer major bug

[Bruno Coudoin]

Le mardi 28 novembre 2006 à 20:24 +0100, Patrice Mandin a écrit :
> Le Mon, 27 Nov 2006 23:24:43 +0100
> Bruno Coudoin <bruno.coudoin at free.fr> a écrit:
> 
> > > I just looked your patch, and this is what I was fearing: I am the
> > > cause. From your patch, it seems the timidity path list is filled
> > > only the first time its init function is called (because of the
> > > static variable I added). So the second time the timidity function
> > > is called, the path list is kept empty with NULL string pointer,
> > > hence the crash.
> > > 
> > You don't need your static variable anymore. With my patch, pathlist
> > is inited and freed at each open/close. The "pathlist = NULL" in
> > free_pathlist() does the job to make sure we reinit it at next open.
> > 
> > > I'll try to fix this as soon as possible.
> > 
> > Excellent new, thank a lot from the GCompris project.
> 
> OK, I tested it today, and it works, so I commited to svn. I also marked
> bugs 295 and 274 (which are likely the same) as FIXED in bugzilla.
> 
> Hum, need better check, because it depends if timidity support was
> compiled in for 274, which I doubt for win32 platform.
> 

Do you plan to make a new release, frankly, that would help us because
we rely on this ?

-- 
Bruno Coudoin
http://gcompris.net Free educational software for kids
http://ofset.org    Free educational software for all
http://toulibre.org Logiciel Libre à Toulouse