Quoting Vittorio Giovara (2016-09-20 23:11:14)
> On Tue, Sep 20, 2016 at 3:05 PM, Anton Khirnov <an...@khirnov.net> wrote:
> > Quoting Vittorio Giovara (2016-09-19 22:33:42)
> >> ---
> >>  libavcodec/avcodec.h | 3 +++
> >>  1 file changed, 3 insertions(+)
> >>
> >> diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h
> >> index c863e53..840222c 100644
> >> --- a/libavcodec/avcodec.h
> >> +++ b/libavcodec/avcodec.h
> >> @@ -1190,6 +1190,9 @@ typedef struct AVCPBProperties {
> >>   * @{
> >>   */
> >>  enum AVPacketSideDataType {
> >> +    /* An AV_PKT_DATA_PALETTE side data packet contains exactly 
> >> AVPALETTE_SIZE
> >
> > Needs an extra *
> >
> >> +     * bytes worth of palette.
> >
> >> Usually the decoder needs to track its content
> >> +     * and set palette_has_changed on the output frame when updated. */
> >
> > This part feels a bit too vague. I'd just say that this side data
> > signals a new palette to decoders and leave it at that.
> >
> > --
> > Anton Khirnov
> > _______________________________________________
> > libav-devel mailing list
> > libav-devel@libav.org
> > https://lists.libav.org/mailman/listinfo/libav-devel
> 
> Would this be OK?

Yes, fine with me.

-- 
Anton Khirnov
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to