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

Reply via email to