[FFmpeg-devel] [PATCH 2/4] avformat/avframe.h: Add a flag in AVIndexEntry to discard frame after decoding.

[Hendrik Leppkes]

On Wed, Aug 10, 2016 at 3:48 AM, Sasi Inguva
<isasi-at-google.com at ffmpeg.org> wrote:
> Signed-off-by: Sasi Inguva <isasi at google.com>
> ---
>  libavformat/avformat.h | 3 +++
>  libavformat/version.h  | 2 +-
>  2 files changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/libavformat/avformat.h b/libavformat/avformat.h
> index d8a6cf3..8cf1401 100644
> --- a/libavformat/avformat.h
> +++ b/libavformat/avformat.h
> @@ -814,6 +814,9 @@ typedef struct AVIndexEntry {
>                                 * is known
>                                 */
>  #define AVINDEX_KEYFRAME 0x0001
> +#define AVINDEX_DISCARD_FRAME  0x0002    /**
> +                                          * Flag is used to indicate which frame should be discarded after decoding.
> +                                          */

Having this flag on the index seems ... odd. If I would expect to get
it from anywhere, it would be the AVPacket the demuxer outputs (which
you add later), but not having to cross-check the index for this.
What purpose does it really server to have it here, and not only on AVPacket?

- Hendrik