On 02/19/2016 10:10 PM, Michael Niedermayer wrote:
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

I haven't the foggiest. Your assumption might be correct, though.

Mats

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

Reply via email to