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

Diego Biurrun diego
Tue Jun 17 10:09:31 CEST 2008

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.


More information about the ffmpeg-devel mailing list