[FFmpeg-devel] [PATCH]lavfi/palettegen: Allow setting the background colour

Carl Eugen Hoyos ceffmpeg at gmail.com
Sun Oct 29 02:47:03 EEST 2017


2017-10-28 23:50 GMT+02:00 Clément Bœsch <u at pkh.me>:
> On Sat, Oct 28, 2017 at 10:57:32PM +0200, Carl Eugen Hoyos wrote:
>> 2017-10-17 23:42 GMT+02:00 Carl Eugen Hoyos <ceffmpeg at gmail.com>:
>>
>> > Attached patch is useful in combination with the transparency patch
>> > for paletteuse.
>>
>> > +    { "background", "set a background color for transparency", OFFSET(background), AV_OPT_TYPE_COLOR, {.str="white"}, CHAR_MIN, CHAR_MAX, FLAGS },
>>
>> Locally changed to "lime" to make sure fate stays unchanged.
>>
>
> Please use "transparency_color" as option name (and field struct).

I liked "background";-(

> Also, the correct type in the structure is uint8_t[4]
>
> And ideally, you would also document the option in doc/filters.texi
>
> You can apply any time with these changes.

Done, thank you!

Carl Eugen


More information about the ffmpeg-devel mailing list