[FFmpeg-devel] [PATCH 2/3] avcodec: use the new event flags field instead of setting AVFrame.palette_has_changed
James Almer
jamrial at gmail.com
Wed Apr 12 14:26:48 EEST 2023
On 4/12/2023 5:09 AM, Anton Khirnov wrote:
> Quoting James Almer (2023-04-11 22:38:50)
>> diff --git a/doc/APIchanges b/doc/APIchanges
>> index 59216231f9..85060768b3 100644
>> --- a/doc/APIchanges
>> +++ b/doc/APIchanges
>> @@ -4,6 +4,7 @@ API changes, most recent first:
>>
>> 2023-04-11 - xxxxxxxxxx - lavc 60.10.100 - avcodec.h
>> Add event_flags to AVCodecContext.
>> + Add AVCTX_EVENT_FLAG_PALETTE_HAS_CHANGED.
>
> I think we should first figure out how is the field supposed to be
> useful to our users. Maybe it's not and we can just remove it.
That's fine. I have no idea if it's actually used or useful (You can
always just memcmp(prev->data[1], cur->data[1], 256) on every frame
after all), but removing without replacement tends to be controversial,
hence why i did this.
More information about the ffmpeg-devel
mailing list