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

Michael Niedermayer michaelni
Tue Aug 4 00:45:16 CEST 2009


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


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

Changelog cant hurt ...

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The greatest way to live with honor in this world is to be what we pretend
to be. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090804/2f2263a6/attachment.pgp>



More information about the ffmpeg-devel mailing list