[FFmpeg-devel] [PATCH] vpxenc: add codec specific speed option

[James Zern]

On Fri, Jan 7, 2011 at 14:33, Michael Niedermayer <michaelni at gmx.at> wrote:
> On Fri, Jan 07, 2011 at 02:26:54PM -0800, James Zern wrote:
>> On Fri, Jan 7, 2011 at 03:16, Vladimir Pantelic <vladoman at gmail.com> wrote:
>> > James Zern wrote:
>> >>
>> >> -speed maps to VP8E_SET_CPUUSED [-16,16].
>> >>
>> >> While this establishes the framework for libvpx codec specific options
>> >> I was wondering about name clashes with these. Is 'speed' too generic
>> >> for this?
>> >
>> > for me "speed" does not sound like a cpu usage option...
>> >
>> This actually affects speed by disabling or replacing components of
>> the encoder, e.g., rd, motion search, loop filter.
>
> why dont you then use the existing options that enable and tune these?
>
The resulting mapping back to a value for this parameter would be more
complex than the pass through as there is no way to directly set the
individual options. It might become confusing if incompatible options
are selected, but inspecting flags/me_method I suppose wouldn't be
impossible.