[MPlayer-dev-eng] [PATCH] Better handling of win32 GUI thread

[Zuxy Meng]

Hi,

2007/11/8, Zuxy Meng <zuxy.meng at gmail.com>:
> Hi,
>
> 2007/11/6, Zuxy Meng <zuxy.meng at gmail.com>:
> > Hi,
> >
> > 2007/11/1, Zuxy Meng <zuxy.meng at gmail.com>:
> > > Hi,
> > >
> > > The attached patch contains two fixes:
> > >
> > > 1. Use _beginthreadex instead of CreateThread to create the thread so
> > > as to avoid a small memory leak when mplayer is linked to MS CRT.
> > >
> > > 2. Remove the call to TerminateThread(). For one thing this isn't the
> > > recommended way to terminate a thread; for another it's called with an
> > > incorrect argument and therefore has always failed: the first
> > > parameter should be a handle to the thread instead of the thread
> > > function. And without this call the GUI thread will terminate itself
> > > anyway once mygui == NULL.
> >
> > Update patch that will free the GUI thread in a cleaner way.
>
> I'll apply this weekend if nobody objects.

Done.
-- 
Zuxy
Beauty is truth,
While truth is beauty.
PGP KeyID: E8555ED6