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

[Robert Krüger]

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 1: Keep mplayer semantics (0 == forever) and keep OPT_INT. That would mean when decrementing the counter to zero, I would add a special case to set it to -1.
>> 
>> option 2: Kick mplayer semantics, keep OPT_INT  and have -1 mean loop forever
>> 
>> option 3: Keep mplayer semantics and kick OPT_INT and make it a function that sets it to -1 when 0 is specified as a value. 
>> 
>> Which one do you prefer?
> 
> not 2
> otherwise what you think is cleaner
> 


updated patch attached.

Regards,

Robert

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