[FFmpeg-devel] [PATCH] remove redundant libvpx AVOptions

[Michael Niedermayer]

On Thu, Jan 05, 2012 at 04:07:24PM -0800, James Zern wrote:
> On Thu, Jan 5, 2012 at 11:47, Michael Niedermayer <michaelni at gmx.at> wrote:
> > On Wed, Jan 04, 2012 at 11:34:50PM +0530, j at v2v.cc wrote:
> >> On 01/04/2012 05:26 PM, Carl Eugen Hoyos wrote:
> >>
> >> >> currently ffmpeg provides 2 ways of setting many libvpx flags.
> >> >> this is confusing. there should be just one way to set those flags.
> >> >> this makes ffmpeg -h much cleaner and documentation and use
> >> >> easier.
> >> >
> >> > Given that is must break (some) scripts, I am slightly against such changes.
> >> >
> >>
> >> as a minimum the double declaration of best, good and realtime should be
> >> removed.
> >
> > fixed, thanks
> >
> >
> >>
> >> doc/encoders.texi also still references vp8flags=+altref but its gone;
> >> so (some) scripts are broken already.
> >
> > yes, sadly.
> > do you volunteer to update the docs or add the option back for
> > compatibility ?
> >
> Both options attached, your preference.

I think maintaining compatibility is slightly preferable, so i applied
that one

Thanks

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

Let us carefully observe those good qualities wherein our enemies excel us
and endeavor to excel them, by avoiding what is faulty, and imitating what
is excellent in them. -- Plutarch
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120106/08fed5b2/attachment.asc>