[FFmpeg-devel] [PATCH] ffplay show status and refresh

Benoit Fouet benoit.fouet
Mon Aug 3 11:27:34 CEST 2009


On 2009-08-03 11:24, Benoit Fouet wrote:
> On 2009-07-31 21:25, Michael Niedermayer wrote:
>> On Fri, Jul 31, 2009 at 12:15:55PM +0200, Benoit Fouet wrote:
>>> On 2009-07-31 09:04, Benoit Fouet wrote:
>>>> On 2009-07-31 01:10, Benoit Fouet wrote:
>>>>> Michael Niedermayer wrote :
>>>>>> [...]
>>>>>>
>>>>>> why dont we make cmdutils handle noX for OPT_BOOL X ?
>>>>>>   
>>>>> something like the attached patch ?
>>>>>
>>>> or rather this one (tested this time :) )
>>>>
>>> well... actually, there was an issue in the previous one when the option
>>> was found without the 'no' but wasn't a bool.
>>> Fixed now, hopefully (sorry for all the noise).
>> no problem, ffmpeg devs are very scatterbrained
>>
>> and patch looks ok
>>
>> [...]
>>
> 
> applied
> 

while I'm at it, I changed back 'stats' option type, ok to apply too ?
another point: how to advertise the new handling noX for OPT_BOOL X ?
Changelog ? or is it just not needed ?

Ben
-------------- next part --------------
A non-text attachment was scrubbed...
Name: nostats.diff
Type: text/x-diff
Size: 947 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090803/857659d6/attachment.diff>



More information about the ffmpeg-devel mailing list