[Mplayer-cvslog] CVS: main/libvo vo_x11.c,1.108,1.109

Alban Bedel albeu at free.fr
Sat May 25 12:16:54 CEST 2002


Hi Arpi,

on Sat, 25 May 2002 12:15:48 +0200 you wrote:

> Hi,
> 
> > Destroy the old win when config is called multiple times
> 
> > - 
> > +     if(vo_window != None) {
> > +       freeMyXImage();
> > +       XUnmapWindow( mDisplay,vo_window );
> > +       XDestroyWindow(mDisplay, vo_window);
> > +     }
> 
> it would be better, if the window is resized and ximages re-created, instead
> of destroying and re-creating the whole window, at the second config()
> 
> anyway i know it is not easy to implement :)
> 
I thought the very same thing but I needed this for the dxr2, so I just added this
quick fix. I'm not 100% sure but I think xv also need a "multiple config" fix.
BTW, just for info when do you plan to really use this feature ? I mean uninit the
vo betwen different files only when needed. 0.90 or the next release ?
	Albeu



More information about the MPlayer-cvslog mailing list