[FFmpeg-devel] [PATCH 1/2] ffplay: factorize frame queue operations

[Marton Balint]

On Wed, 17 Sep 2014, Michael Niedermayer wrote:

> On Wed, Sep 17, 2014 at 03:47:09AM +0200, Michael Niedermayer wrote:
>> On Wed, Sep 17, 2014 at 12:19:49AM +0200, Marton Balint wrote:
>>>
>>> On Sat, 6 Sep 2014, Marton Balint wrote:
>>>
>>>> Create a generic frame and a frame queue struct to handle video picture queues
>>>> and subtitle picture queues with common code. Also add the possibility to queue
>>>> AVFrames, however at the moment we only use SDL_Overlay buffers for video and
>>>> AVSubtitles for subtitles.
>>>>
>>>> Signed-off-by: Marton Balint <cus at passwd.hu>
>>>> ---
>>>> ffplay.c | 357 +++++++++++++++++++++++++++++++++++----------------------------
>>>> 1 file changed, 199 insertions(+), 158 deletions(-)
>>>>
>>>> diff --git a/ffplay.c b/ffplay.c
>>>> index 833b5b1..c59065f 100644
>>>
>>> Hello Michael,
>>>
>>> Please merge from my stable branch for the ffplay patches:
>>>
>>> 5a5128b ffplay: factorize frame queue operations
>>> 1643764 ffplay: use frame queue to determine last used pos
>>
>> merged
>
> Seems this causes problems on apple:
> http://fate.ffmpeg.org/report.cgi?time=20140917022057&slot=x86_64-darwin-gcc-4.9
>
> ill commit something that might fix the build in a moment so its not
> left broken but please check it when you have time
>

Your fix is OK, thanks, and sorry for the hiccup.

Regards,
Marton