Hi,

On Sat, Feb 11, 2012 at 12:25 PM, Anton Khirnov <[email protected]> wrote:
> ---
>  libavcodec/dnxhdenc.c |   19 ++++++++++++-------
>  1 files changed, 12 insertions(+), 7 deletions(-)
[..]
> +    pkt->flags |= AV_PKT_FLAG_KEY;

I vaguely recall that we had some code in libavformat/utils.c that set
this flag for each keyframe-only codec, and we had a utility flag or
function that knows which codecs are kf-only. We could use that here
also, so that the flag doesn't have to be set manually by
keyframe-only codecs.

Then again I don't care much, since that may actually have been
intended as a hack for demuxers that fail to set the flag correctly
because of bitstream bugs, so if you'd prefer to ignore this, ignore
it and LGTM.

Ronald
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to