[FFmpeg-devel] [PATCH] Add loop option to ffplay

[Robert Krüger]

On 23.03.2010, at 08:19, Robert Kr?ger wrote:

> 
> On 23.03.2010, at 01:06, Michael Niedermayer wrote:
> 
>> On Sun, Mar 21, 2010 at 08:35:55PM +0100, Robert Kr?ger wrote:
>>> 
>>> On 21.03.2010, at 18:55, Michael Niedermayer wrote:
>>> 
>>>> On Sun, Mar 21, 2010 at 06:47:43PM +0100, Robert Kr?ger wrote:
>>>>> 
>>>>> On 21.03.2010, at 16:51, Michael Niedermayer wrote:
>>>>> 
>>>>>> On Sat, Mar 20, 2010 at 01:58:08PM +0100, Robert Kr?ger wrote:
>>>>>>> 
>>>>>>> On 20.03.2010, at 12:54, Stefano Sabatini wrote:
>>>>>>> 
>>>>>>>> On date Saturday 2010-03-20 12:08:32 +0100, Stefano Sabatini encoded:
>>>>>>>>> On date Saturday 2010-03-20 08:53:16 +0100, Robert Kr?ger encoded:
>>>>>>>>>> 
>>>>>>>>>> On 19.03.2010, at 21:07, Michael Niedermayer wrote:
>>>>>>>> [...]
>>>>> option 
>> 
>> hmmm, thats quite a bit more complex than just reading as is
>> and
>> 
>> if(loop<0 || (loop && !--loop))
>> 
> 
> 
> updated patch attached.
> 
> Regards,
> 
> Robert
> 

wait, don't review! It's now broken without autoexit. Update below. I'm probably blind but I didn't see how to differentiate between loop being zero because it was specified as that on the command line and because it was decremented in the conditional, which is why I added the additional if to make it work again.

Regards,

Robert

-------------- next part --------------
A non-text attachment was scrubbed...
Name: ffplay-loop-option.patch
Type: application/octet-stream
Size: 2179 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100323/8013bf12/attachment.obj>
-------------- next part --------------