[FFmpeg-devel] [PATCH] FFplay speedup if stepping whilst paused

Benoit Fouet benoit.fouet
Mon Aug 27 10:07:36 CEST 2007


Michael Niedermayer wrote:
> Hi
>
> On Sun, Aug 26, 2007 at 01:38:12PM +0200, Stefano Sabatini wrote:
>   
>> On date Sunday 2007-08-26 13:34:44 +0200, Stefano Sabatini encoded:
>>     
>>> On date Sunday 2007-08-26 13:15:08 +0200, Michael Niedermayer encoded:
>>>       
>>>> Hi
>>>>
>>>> On Sun, Aug 26, 2007 at 11:45:22AM +0200, Stefano Sabatini wrote:
>>>> [...]
>>>>         
>>>>> Since Craig does not seem to come back with a new patch I've done
>>>>> that.
>>>>>
>>>>> I simply removed the line:
>>>>> cur_stream->video_current_pts = get_video_clock(cur_stream);
>>>>>
>>>>> since that is already done in stream_pause(), as suggested by Michael.
>>>>>
>>>>> BTW the patch should be attributed anyway to Craig in the commit
>>>>> message.
>>>>>           
>>>> [...]
>>>>         
>>>>> Index: ffplay.c
>>>>> ===================================================================
>>>>> --- ffplay.c	(revision 10230)
>>>>> +++ ffplay.c	(working copy)
>>>>> @@ -2196,9 +2196,9 @@
>>>>>  static void step_to_next_frame(void)
>>>>>  {
>>>>>      if (cur_stream) {
>>>>> -        if (cur_stream->paused)
>>>>>           
>>>> [...]
>>>>         
>>>>> +       if (cur_stream->paused)
>>>>>           
>>>> no
>>>>         
>>> Oops, sorry.
>>> New try attached.
>>>       
>> New oops and new sorry, this time I hope I'm really going to attach
>> it.
>>     
>
> looks ok
>
> [...]
>   

Applied (with credits to Craig)

-- 
Ben
Purple Labs S.A.
www.purplelabs.com




More information about the ffmpeg-devel mailing list