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

[Carl Eugen Hoyos]

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

Carl Eugen