[MPlayer-cvslog] r33363 - trunk/mplayer.c

[Reimar Döffinger]

On Thu, May 05, 2011 at 12:14:02AM +0200, Clément Bœsch wrote:
> On Wed, May 04, 2011 at 10:13:26PM +0200, reimar wrote:
> > Author: reimar
> > Date: Wed May  4 22:13:25 2011
> > New Revision: 33363
> > 
> > Log:
> > Move freeing of subtitle data to uninit_player so it not only happens
> > on switching files but also when exiting.
> > The makes it easier to debug memleaks with valgrind.
> > 
> > Modified:
> >    trunk/mplayer.c
> > 
> > Modified: trunk/mplayer.c
> > ==============================================================================
> > --- trunk/mplayer.c	Wed May  4 17:16:35 2011	(r33362)
> > +++ trunk/mplayer.c	Wed May  4 22:13:25 2011	(r33363)
> > @@ -637,6 +637,29 @@ void uninit_player(unsigned int mask){
> >      getch2_disable();
> >    }
> >  
> > +  if (mask & INITIALIZED_SUBS) {
> 
> I think you forgot to define INITIALIZED_SUBS…

More I forgot to commit the mp_core.h change that added.
For once I actually did test before committing :-)