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

[Torge Matthies (＠tmatthies)]

On Tue Jan 10 01:42:11 2023 +0000, **** wrote:
> Zebediah Figura replied on the mailing list:
> On 1/9/23 19:24, Torge Matthies wrote:
> > ICM_DECOMPRESS_GET_FORMAT does not work well with the BITMAPINFOHEADER
> from MS-CRAM, because a biBitCount of 24
> > (or anything above 8) indicates RGB555 encoding, which actually means
> the bit count is 16 bpp, but the decoder rejects
> > any format that doesn't have biBitCount of exactly 8 or 16.
> I don't understand this patch description. biBitCount of 24 doesn't mean 
> 555, it means 24-bit RGB, at least if BI_RGB is specified. What input 
> format (compression and bit depth) are we passing to 
> ICM_DECOMPRESS_GET_FORMAT? And are we sure that there isn't a bug in the 
> decoder?

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