[FFmpeg-devel] [PATCH 1/2] ffplay: avoid window resize crash on osx with libsdl 1.2.14

[Michael Niedermayer]

On Sun, Oct 16, 2011 at 08:26:06PM +0200, Marton Balint wrote:
> 
> On Wed, 12 Oct 2011, Carl Eugen Hoyos wrote:
> 
> >Jean First <jeanfirst <at> gmail.com> writes:
> >
> >>+++ b/ffplay.c
> >>@@ -947,13 +947,7 @@ static int video_open(VideoState *is){
> >>     if(screen && is->width == screen->w && screen->w == w
> >>        && is->height== screen->h && screen->h == h)
> >>         return 0;
> >>-
> >>-#ifndef __APPLE__
> >>     screen = SDL_SetVideoMode(w, h, 0, flags);
> >>-#else
> >>-    /* setting bits_per_pixel = 0 or 32 causes blank video on OS X */
> >>-    screen = SDL_SetVideoMode(w, h, 24, flags);
> >>-#endif
> >
> >This patch causes a double-free on exit in SDL_FreeYUVOverlay() for me on PPC
> >oSX 10.5, but since it fixes wrong colours in many samples, I am all for it.
> 
> Applied to my stable branch, thanks.
> 
> Michael, please merge from ffplay stable for the following patches:
> 
> 72776ad ffplay: avoid window resize crash on osx with libsdl 1.2.14
> abb0e4f ffplay: add delay multiple times to frame_timer if it is less than current time
> d2d8e1e ffplay: remove early frame drop functionality
> 4e268aa ffplay: calculate target clock dynamically, make code more readable

merged, thanks

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Breaking DRM is a little like attempting to break through a door even
though the window is wide open and the only thing in the house is a bunch
of things you dont want and which you would get tomorrow for free anyway
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20111016/80f6388f/attachment.asc>