[x264-devel] [Patch][RFC] moving presets, tunings, profiles from x264.c to common.c

Jason Garrett-Glaser darkshikari at gmail.com
Tue Nov 10 12:03:07 CET 2009


On Tue, Nov 10, 2009 at 2:55 AM, aviad rozenhek <aviadr1 at gmail.com> wrote:
>>
>>
>> The problem is that this changes the way the presets work.  Currently:
>>
>> preset is applied
>>
>> tune is applied
>>
>> all other settings are applied
>>
>> profile is applied
>>
>> Your method makes them applied in commandline order, which may confuse
>> some users (especially with regard to --profile).  It would make more
>> sense, IMO, to have param_default() take a preset and tuning as an
>> argument, for example.
>>
>> Dark Shikari
>
> I left the logic in x264.c intact, so preset, tune. other_settings,
>  profiles are still applied in order.
> so they are not applied in commandline order.

Yes, but the order will still be different when used from *other*
applications, which will only serve to confuse users.

Dark Shikari


More information about the x264-devel mailing list