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? --- 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 + * bytes worth of palette.This side data signals that a new palette is preset. */ AV_PKT_DATA_PALETTE, /** -- 2.10.0 -- Vittorio _______________________________________________ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel