[FFmpeg-devel] [PATCH 2/3] lavf/avienc: Write palette after BITMAPINFOHEADER

Michael Niedermayer michael at niedermayer.cc
Fri Feb 19 22:10:47 CET 2016


On Fri, Feb 19, 2016 at 10:05:32PM +0100, Mats Peterson wrote:
> On 02/19/2016 10:00 PM, Michael Niedermayer wrote:
> >>>>-
> >>>>                  pc_tag = ff_start_tag(pb, tag);
> >>>>                  avio_w8(pb, 0);
> >
> >>>>                  avio_w8(pb, 0);
> >
> >>>>                  avio_wl16(pb, 0); // reserved
> >>>>-                for (i = 0; i<256; i++) {
> >>>>+                for (i = 0; i < pal_size; i++) {
> >>>
> >>>this mismatches the stored size above
> >>
> >>Mismatches what stored size?
> >
> >the "avio_w8(pb, 0);" above
> >
> 
> Right. I'll just set that one to 1 << bits_per_coded_sample then.
> Funny that not even Media Player supports these palette change
> chunks, by the way.

what does it do with files from other sources that contain such
chunks ?
maybe it supports them for only some vieo codecs but not rawvideo

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I am the wisest man alive, for I know one thing, and that is that I know
nothing. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160219/cf111317/attachment.sig>


More information about the ffmpeg-devel mailing list