Hi
Bobby Bingham wrote:
> On Tue, 16 Oct 2007 21:18:15 +0200 (CEST)
> vitor <[EMAIL PROTECTED]> wrote:
>
>> Log:
>> Fix crash after video end.
>> As a positive secondary effect, I think the new patch #01 will be
>> easier to get in the trunk.
>>
>> Please flame me if I'm doing anything wrong.
>
> [...]
>
>> Modified: libavfilter/diffs/01_ffplay_get_video_frame.diff
>> ==============================================================================
>> --- libavfilter/diffs/01_ffplay_get_video_frame.diff (original)
>> +++ libavfilter/diffs/01_ffplay_get_video_frame.diff Tue Oct
>> 16 21:18:14 2007 @@ -2,7 +2,7 @@ Index: ffplay.c
>> ===================================================================
>> --- ffplay.c (revision 10755)
>> +++ ffplay.c (working copy)
>> -@@ -1341,21 +1341,12 @@
>> +@@ -1341,49 +1341,63 @@
>> avcodec_default_release_buffer(c, pic);
>> }
>>
>> @@ -23,11 +23,15 @@ Index: ffplay.c
>> - if (packet_queue_get(&is->videoq, pkt, 1) < 0)
>> - break;
>> -
>> -+ while (packet_queue_get(&is->videoq, pkt, 1) >= 0) {
>> ++ if (packet_queue_get(&is->videoq, pkt, 1) < 0)
>> ++ return -1;
>> ++
>
> This change breaks video playback for me. Changing it back to a while
> loop, but keeping all the other changes seems to fix it for me, and
> still fixes the hang at the end of playback.
Thanks, I've found a sample that hangs with current code to test. I
found a way to fix it in patch #03. I don't like the while loop because
I want to keep patch #01 with no functionality changes...
-Vitor
_______________________________________________
FFmpeg-soc mailing list
[email protected]
http://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-soc
