[FFmpeg-devel] [PATCH] Cosmetics: display generic advanced options just after generic non-advanced options

Stefano Sabatini stefano.sabatini-lala
Tue Jun 17 12:13:12 CEST 2008


On date Tuesday 2008-06-17 10:09:31 +0200, Diego Biurrun encoded:
> On Tue, Jun 17, 2008 at 09:30:36AM +0200, Stefano Sabatini wrote:
> > On date Monday 2008-06-16 17:21:20 +0200, Diego Biurrun encoded:
> > > On Mon, Jun 16, 2008 at 03:11:06PM +0200, Stefano Sabatini wrote:
> > > > Hi, as in subject, looks more consistent with order followed by the
> > > > following option printing (that is: non advanced, video options,
> > > > advanced video options, non advanced audio options, advanced audio
> > > > options, ...).
> > > 
> > > Looks OK to me.
> > 
> > Is it OK to apply or do I have to wait for Michael's approval?
> 
> In theory you have to wait for Michael's approval.  Another developer
> agreeing with the patch does of course give you some assurance that you
> are going down the right way.
> 
> Michael said he does not care about these patches and wanted somebody
> else to review them.  So in this case, I would say go ahead and apply.

Thanks, applied.
-- 
FFmpeg = Fast & Fress MultiPurpose EnGraver




More information about the ffmpeg-devel mailing list