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

Steven Liu lq at chinaffmpeg.org
Sat Sep 14 05:02:47 EEST 2019



> 在 2019年9月13日,19:53,Jun Zhao <mypopydev at gmail.com> 写道:
> 
> From: Jun Zhao <barryjzhao at tencent.com>
> 
> refine the log message, it's will help the debugging
> 
> Signed-off-by: Jun Zhao <barryjzhao at 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 at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".


patchset LGTM


Thanks
Steven







More information about the ffmpeg-devel mailing list