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

[Zuxy Meng]

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.

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