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

Clément Bœsch u at pkh.me
Sun Oct 29 00:50:35 EEST 2017


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).

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.

Thanks,

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20171028/b83d3d17/attachment.sig>


More information about the ffmpeg-devel mailing list