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
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
