diff mbox

[FFmpeg-devel,V1,1/4] lavf/hls: refine the log message

Message ID 1568375617-19090-1-git-send-email-mypopydev@gmail.com
State Accepted
Commit 48d96ce29a2dc459bc844e284322c6bd00cd3f2f
Headers show

Commit Message

Jun Zhao Sept. 13, 2019, 11:53 a.m. UTC
From: Jun Zhao <barryjzhao@tencent.com>

refine the log message, it's will help the debugging

Signed-off-by: Jun Zhao <barryjzhao@tencent.com>
---
 libavformat/hls.c |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

Comments

Liu Steven Sept. 14, 2019, 2:02 a.m. UTC | #1
> 在 2019年9月13日,19:53,Jun Zhao <mypopydev@gmail.com> 写道:
> 
> From: Jun Zhao <barryjzhao@tencent.com>
> 
> refine the log message, it's will help the debugging
> 
> Signed-off-by: Jun Zhao <barryjzhao@tencent.com>
> ---
> libavformat/hls.c |    6 +++---
> 1 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/libavformat/hls.c b/libavformat/hls.c
> index 61b6759..10fc568 100644
> --- a/libavformat/hls.c
> +++ b/libavformat/hls.c
> @@ -664,7 +664,7 @@ static int open_url(AVFormatContext *s, AVIOContext **pb, const char *url,
>         } else if (ret < 0) {
>             if (ret != AVERROR_EOF)
>                 av_log(s, AV_LOG_WARNING,
> -                    "keepalive request failed for '%s' when opening url, retrying with new connection: %s\n",
> +                    "keepalive request failed for '%s' with error: '%s' when opening url, retrying with new connection\n",
>                     url, av_err2str(ret));
>             ret = s->io_open(s, pb, url, AVIO_FLAG_READ, &tmp);
>         }
> @@ -721,7 +721,7 @@ static int parse_playlist(HLSContext *c, const char *url,
>         } else if (ret < 0) {
>             if (ret != AVERROR_EOF)
>                 av_log(c->ctx, AV_LOG_WARNING,
> -                    "keepalive request failed for '%s' when parsing playlist, retrying with new connection: %s\n",
> +                    "keepalive request failed for '%s' with error: '%s' when parsing playlist\n",
>                     url, av_err2str(ret));
>             in = NULL;
>         }
> @@ -1483,7 +1483,7 @@ reload:
>         if (ret < 0) {
>             if (ff_check_interrupt(c->interrupt_callback))
>                 return AVERROR_EXIT;
> -            av_log(v->parent, AV_LOG_WARNING, "Failed to open segment %d of playlist %d\n",
> +            av_log(v->parent, AV_LOG_WARNING, "Failed to open next segment %d of playlist %d\n",
>                    v->cur_seq_no + 1,
>                    v->index);
>         } else {
> -- 
> 1.7.1
> 
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".


patchset LGTM


Thanks
Steven
diff mbox

Patch

diff --git a/libavformat/hls.c b/libavformat/hls.c
index 61b6759..10fc568 100644
--- a/libavformat/hls.c
+++ b/libavformat/hls.c
@@ -664,7 +664,7 @@  static int open_url(AVFormatContext *s, AVIOContext **pb, const char *url,
         } else if (ret < 0) {
             if (ret != AVERROR_EOF)
                 av_log(s, AV_LOG_WARNING,
-                    "keepalive request failed for '%s' when opening url, retrying with new connection: %s\n",
+                    "keepalive request failed for '%s' with error: '%s' when opening url, retrying with new connection\n",
                     url, av_err2str(ret));
             ret = s->io_open(s, pb, url, AVIO_FLAG_READ, &tmp);
         }
@@ -721,7 +721,7 @@  static int parse_playlist(HLSContext *c, const char *url,
         } else if (ret < 0) {
             if (ret != AVERROR_EOF)
                 av_log(c->ctx, AV_LOG_WARNING,
-                    "keepalive request failed for '%s' when parsing playlist, retrying with new connection: %s\n",
+                    "keepalive request failed for '%s' with error: '%s' when parsing playlist\n",
                     url, av_err2str(ret));
             in = NULL;
         }
@@ -1483,7 +1483,7 @@  reload:
         if (ret < 0) {
             if (ff_check_interrupt(c->interrupt_callback))
                 return AVERROR_EXIT;
-            av_log(v->parent, AV_LOG_WARNING, "Failed to open segment %d of playlist %d\n",
+            av_log(v->parent, AV_LOG_WARNING, "Failed to open next segment %d of playlist %d\n",
                    v->cur_seq_no + 1,
                    v->index);
         } else {