On Fri,  1 Aug 2014 13:53:41 +0200
Luca Barbato <[email protected]> wrote:

> And leverage the video index if the video is just disabled as wm4
> did in an initial patch.
> ---
>  libavformat/flvdec.c | 7 +++++--
>  1 file changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/libavformat/flvdec.c b/libavformat/flvdec.c
> index 8e4cc5e..034e346 100644
> --- a/libavformat/flvdec.c
> +++ b/libavformat/flvdec.c
> @@ -818,6 +818,11 @@ skip:
>              st = create_stream(s, is_audio ? AVMEDIA_TYPE_AUDIO
>                                             : AVMEDIA_TYPE_VIDEO);
>          av_dlog(s, "%d %X %d \n", is_audio, flags, st->discard);
> +
> +        if ((flags & FLV_VIDEO_FRAMETYPE_MASK) == FLV_FRAME_KEY ||
> +            is_audio)
> +            av_add_index_entry(st, pos, dts, size, 0, AVINDEX_KEYFRAME);
> +
>          if ((st->discard >= AVDISCARD_NONKEY &&
>               !((flags & FLV_VIDEO_FRAMETYPE_MASK) == FLV_FRAME_KEY || 
> is_audio)) ||
>              (st->discard >= AVDISCARD_BIDIR &&
> @@ -826,8 +831,6 @@ skip:
>              avio_seek(s->pb, next, SEEK_SET);
>              continue;
>          }
> -        if ((flags & FLV_VIDEO_FRAMETYPE_MASK) == FLV_FRAME_KEY)
> -            av_add_index_entry(st, pos, dts, size, 0, AVINDEX_KEYFRAME);
>          break;
>      }
>  

I agree with this patch. (Just makes me wonder why the code did what it
did.)
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to