Re: [PATCH 0/1] MR1929: comctl32: Use ICGetDisplayFormat to open the AVI codec and find the output format

[Zebediah Figura (＠zfigura)]

> `biBitCount` of 24 apparently means RGB555 in MS-CRAM, and gstreamer agrees (see the gst-discoverer-1.0 output in the bug report), and the data comes directly from the `strf` header of the AVI file.

Okay, I had to look up how Video-1 actually works to understand what this means. For the benefit of anyone else reading this, Video-1 basically works by quantizing 4x4 blocks into a maximum of 8 distinct colors, but the colors themselves retain the full depth of the "decoded" format, which is either 8-bit palettized RGB, or 16-bit RGB 5/5/5.

Anyway, are we sure this isn't a bug in the decoder? Should msvidc32 allow bit counts other than 8 and 16? It certainly seems suspicious that it cares in ICM_DECOMPRESS_GET_FORMAT but not any other functions.