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

Benoit Fouet benoit.fouet
Tue Aug 4 09:27:09 CEST 2009


On 2009-08-04 00:45, Michael Niedermayer wrote:
> On Mon, Aug 03, 2009 at 11:27:34AM +0200, Benoit Fouet wrote:
>> 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 ?
> 
> iam not against it
> 

Baptiste ?

> 
>> another point: how to advertise the new handling noX for OPT_BOOL X ?
>> Changelog ? or is it just not needed ?
> 
> Changelog cant hurt ...
> 

here is a patch for that:

Index: Changelog
===================================================================
--- Changelog   (revision 19572)
+++ Changelog   (working copy)
@@ -30,6 +30,7 @@ version <next>:
 - Experimental AAC encoder
 - RTP depacketization of ASF and RTSP from WMS servers
 - RTMP support in libavformat
+- noX handling for OPT_BOOL X options




More information about the ffmpeg-devel mailing list