[FFmpeg-devel] [PATCH] avformat/avienc: Store pal8 palette

Michael Niedermayer michael at niedermayer.cc
Fri Feb 19 15:46:16 CET 2016


On Fri, Feb 19, 2016 at 02:15:01PM +0100, Mats Peterson wrote:
> On 02/19/2016 02:08 PM, Mats Peterson wrote:
> >On 02/19/2016 02:05 PM, Mats Peterson wrote:
> >>On 02/18/2016 11:07 PM, Mats Peterson wrote:
> >>>On 02/18/2016 11:04 PM, Michael Niedermayer wrote:
> >>>>On Thu, Feb 18, 2016 at 10:40:42PM +0100, Mats Peterson wrote:
> >>>>>On 02/18/2016 10:27 PM, Mats Peterson wrote:
> >>>>>>On 02/18/2016 09:14 PM, Michael Niedermayer wrote:
> >>>>>>>This can be made more efficient, but first and the main goal of this
> >>>>>>>change is to
> >>>>>>>store it at all
> >>>>>>Great, Michael. I'll try it out.
> >>>>>>
> >>>>>>Mats
> >>>>>>
> >>>>>
> >>>>>I notice that the palette inclusion is only made by using 'xxpc'
> >>>>>chunks in the video data so far. I'm not so sure if these in-data
> >>>>>palette changing chunks are widely supported. Therefore, first and
> >>>>>foremost, and the most widely supported way in my book, is to add an
> >>>>>"initial" palette after the BITMAPINFOHEADER in the strf chunk.
> >>>>>That's where I found it to be a bit tricky, but I'm sure you can
> >>>>>solve it in some way.
> >>>>
> >>>>its quite easy to implement if you want to try
> >>>>just leave some space where the palette should go whe writing the
> >>>>headers and keep track of the location, then when the first
> >>>>packet comes in take its palette and store it there
> >>>>
> >>>
> >>>I could try, yes. But not tonight, I'm a bit tired :) I'll look into it,
> >>>perhaps I can make it.
> >>>
> >>>Mats
> >>
> >>
> >>Will you apply your latest xxpc writing patches to the repository,
> >>Michael? I noticed you haven't done it yet.
> >>
> >>Mats
> >
> >The stuff with CONTAINS_PAL etc.
> >
> 
> I forgot that I had added them before starting to work on this. It
> would be nice if you could apply them "officially" before I
> continue, since they work well.

applied

thanks

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

You can kill me, but you cannot change the truth.
-------------- 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/f52a7cc2/attachment.sig>


More information about the ffmpeg-devel mailing list