diff mbox

[FFmpeg-devel] avformat/flvdec: add debug message to list keyframes index metadata

Message ID 20161104080814.13621-1-lq@chinaffmpeg.org
State Superseded
Headers show

Commit Message

Liu Steven Nov. 4, 2016, 8:08 a.m. UTC
when parsing keyframe index metadata, list the message by debug log

Signed-off-by: Steven Liu <lq@chinaffmpeg.org>
---
 libavformat/flvdec.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

Comments

Michael Niedermayer Nov. 4, 2016, 11:26 a.m. UTC | #1
On Fri, Nov 04, 2016 at 04:08:14PM +0800, Steven Liu wrote:
> when parsing keyframe index metadata, list the message by debug log
> 
> Signed-off-by: Steven Liu <lq@chinaffmpeg.org>
> ---
>  libavformat/flvdec.c |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/libavformat/flvdec.c b/libavformat/flvdec.c
> index e53c345..e2a4669 100644
> --- a/libavformat/flvdec.c
> +++ b/libavformat/flvdec.c
> @@ -114,6 +114,8 @@ static void add_keyframes_index(AVFormatContext *s)
>  
>      if (stream->nb_index_entries == 0) {
>          for (i = 0; i < flv->keyframe_count; i++) {
> +            av_log(s, AV_LOG_DEBUG, "keyframe filepositions = %"PRId64" times = %"PRId64"\n",
> +                   flv->keyframe_filepositions[i], flv->keyframe_times[i] * 1000);

i think this should be AV_LOG_TRACE

[...]
Steven Liu Nov. 4, 2016, 11:56 a.m. UTC | #2
2016-11-04 19:26 GMT+08:00 Michael Niedermayer <michael@niedermayer.cc>:

> On Fri, Nov 04, 2016 at 04:08:14PM +0800, Steven Liu wrote:
> > when parsing keyframe index metadata, list the message by debug log
> >
> > Signed-off-by: Steven Liu <lq@chinaffmpeg.org>
> > ---
> >  libavformat/flvdec.c |    2 ++
> >  1 files changed, 2 insertions(+), 0 deletions(-)
> >
> > diff --git a/libavformat/flvdec.c b/libavformat/flvdec.c
> > index e53c345..e2a4669 100644
> > --- a/libavformat/flvdec.c
> > +++ b/libavformat/flvdec.c
> > @@ -114,6 +114,8 @@ static void add_keyframes_index(AVFormatContext *s)
> >
> >      if (stream->nb_index_entries == 0) {
> >          for (i = 0; i < flv->keyframe_count; i++) {
> > +            av_log(s, AV_LOG_DEBUG, "keyframe filepositions = %"PRId64"
> times = %"PRId64"\n",
> > +                   flv->keyframe_filepositions[i],
> flv->keyframe_times[i] * 1000);
>
> i think this should be AV_LOG_TRACE
>
ok, update soon

>
> [...]
> --
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>
> Democracy is the form of government in which you can choose your dictator
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
>
diff mbox

Patch

diff --git a/libavformat/flvdec.c b/libavformat/flvdec.c
index e53c345..e2a4669 100644
--- a/libavformat/flvdec.c
+++ b/libavformat/flvdec.c
@@ -114,6 +114,8 @@  static void add_keyframes_index(AVFormatContext *s)
 
     if (stream->nb_index_entries == 0) {
         for (i = 0; i < flv->keyframe_count; i++) {
+            av_log(s, AV_LOG_DEBUG, "keyframe filepositions = %"PRId64" times = %"PRId64"\n",
+                   flv->keyframe_filepositions[i], flv->keyframe_times[i] * 1000);
             av_add_index_entry(stream, flv->keyframe_filepositions[i],
                 flv->keyframe_times[i] * 1000, 0, 0, AVINDEX_KEYFRAME);
         }