[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