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

Hendrik Leppkes h.leppkes at gmail.com
Sun Apr 14 21:29:21 EEST 2019


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.

- Hendrik


More information about the ffmpeg-devel mailing list