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

[Robert Krüger]

On 07.01.2011, at 03:36, Michael Niedermayer wrote:

> On Thu, Jan 06, 2011 at 05:43:53PM -0800, 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?
> 
> well, i cant gurantee that speed wont be used for something ffmpeg wide one day
> until then i am fine with you useing speed.
> 

I'm not sure it this is also the case for mencoder but I was under the impression that you were trying to name things that both exist in mplayer and ffmpeg consistently, if the name used in mplayer makes sense and does not conflict with anything in ffmpeg. I think quite a few people are hoping for the -speed <factor> option of mencoder (adjust the speed of the file by adjusting video fps and resampling audio, I think) to make it into ffmpeg one day.

Maybe this is not an issue if it is going to be implemented as a filter or a set of filters anyway but I thought I should just bring this up so you can make a conscious decision for or against that potential name clash, which might not be that far away.

Regards,

Robert