[FFmpeg-devel] [PATCH v5] avformat/flvenc: add add_keyframe_index option

[Steven Liu]

2016-11-08 21:54 GMT+08:00 Michael Niedermayer <michael at niedermayer.cc>:

> On Tue, Nov 08, 2016 at 07:43:49PM +0800, Steven Liu wrote:
> > Add keyframe index metadata
> > Used to facilitate seeking; particularly for HTTP pseudo streaming.
> >  1. read live streaming or file by sequence
> >  2. if use add_keyframe_index option, add a mark flag at the position,
> >     use to insert new context at the last step.
> >  3. add the keyframes *offset* and *timestamp* into a list
> >  4. if use add_keyframe_index option, shift the metadata data from
> >     mark flag offset
> >  5. insert the keyframes *offset* and *timestamp* from the list by
> >     sequence
> >  6. free the list
> >  7. end.
> >
> > Add FATE test case;
> >
> > Reviewed-by: Lou Logan <lou at lrcd.com>
> > Signed-off-by: Steven Liu <liuqi at gosun.com>
> > Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> > ---
> >  doc/muxers.texi                       |    3 +
> >  libavformat/flvenc.c                  |  331
> ++++++++++++++++++++++++++++++++-
> >  tests/Makefile                        |    1 +
> >  tests/fate-run.sh                     |    4 +
> >  tests/fate/flvenc.mak                 |   11 +
> >  tests/ref/fate/flv-add_keyframe_index |   12 ++
> >  6 files changed, 352 insertions(+), 10 deletions(-)
> >  create mode 100644 tests/fate/flvenc.mak
> >  create mode 100644 tests/ref/fate/flv-add_keyframe_index
>
> LGTM
>
> thx
>
> [...]
> --
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>
> The educated differ from the uneducated as much as the living from the
> dead. -- Aristotle
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
>
Applied!

Thanks!