[FFmpeg-devel] [PATCH] avcodec: add drop_changed_frames

James Almer jamrial at gmail.com
Sun Apr 14 21:47:10 EEST 2019

On 4/14/2019 3:29 PM, Hendrik Leppkes wrote:
> On Sun, Apr 14, 2019 at 6:50 PM Gyan <ffmpeg at gyani.pro> wrote:
>> Implemented this patch
>> http://www.ffmpeg.org/pipermail/ffmpeg-devel/2019-March/241733.html
>>   in libavcodec as suggested by Michael
> This sure adds a lot of additional fields to the main struct for a
> rather specialized feature, that I personally rather see in the hands
> of the user of avcodec, not avcodec itself.
> In any case, can't we do this without any new public fields at all?
> Put the initial_* state fields into an internal struct (ie.
> AVCodecInternal), and expose enabling this through
> AVCodecContext->flags or flags2?
> That would make me feel much less dirty looking at this patch, personally.


There has been work to turn public AVCodecContext fields into internal
codec options recently as they were too specialized. This commit adds
half a dozen of such fields in one go, so it just feels wrong.

More information about the ffmpeg-devel mailing list