[FFmpeg-devel] [PATCH] Add Win32 GDI-based screen grabbing

Roger Pack rogerdpack2 at gmail.com
Sat Mar 22 00:58:48 CET 2014


On 12/2/13, Carl Eugen Hoyos <cehoyos at ag.or.at> wrote:
> Calvin Walton <calvin.walton <at> kepstin.ca> writes:
>
>> +    switch (bpp) {
>> +    case 8:
>> +        input_pixfmt = PIX_FMT_PAL8;
>> +        /* TODO: read the palette, respond to changes. */
>
> If it does not work, please remove it.
> In case you don't know what to do with
> the palette, see the last hunk of f60c3a7c
>
>> +    case 32:
>> +        input_pixfmt = PIX_FMT_RGB32;
>
> This should be FMT_BGR0 assuming Windows does
> not support transparency on screen grabbing
> and all pixel formats have an AV_ prefix now,
> so AV_PIX_FMTBGR0

I'll take over and go ahead and do these small modifications next week
sometime if there's no objection (don't want it to get stuck in the
ML).
Thanks!


More information about the ffmpeg-devel mailing list