On Fri, Mar 11, 2016 at 5:10 AM, Mats Peterson
<matsp888-at-yahoo....@ffmpeg.org> wrote:
>
> Regarding the biClrUsed field, I'm setting it to 1 <<
> bits_per_coded_sample if palettized video, since setting it to 0 is
> another case where it won't work with Windows Media Player and Microsoft
> Video 1 in 8 bpp mode.
>

This is not "spec compliance" then, because the spec specifically says
that when its 0, it should be assumed equal to 2^biBitCount, which is
the same value you are explicitly writing now.
Its a redundant value to work around the inability of some players,
not spec compliance - the existing value is perfectly compliant.
Please clarify as much in any potential commit message.

- Hendrik
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to