[FFmpeg-devel] [PATCH v2] lavf/mp3dec: avoid printing useless message in default log level

[wm4]

On Tue,  8 Mar 2016 16:54:42 +0100
Moritz Barsnick <barsnick at gmx.net> wrote:

> "Skipping 0 bytes of junk" is useless to the user, and essentially
> indicates a NOP. At 0 bytes, this message is now pushed back to
> the verbose log level.
> 
> Signed-off-by: Moritz Barsnick <barsnick at gmx.net>
> ---
>  libavformat/mp3dec.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavformat/mp3dec.c b/libavformat/mp3dec.c
> index adc3d2a..672d643 100644
> --- a/libavformat/mp3dec.c
> +++ b/libavformat/mp3dec.c
> @@ -388,7 +388,7 @@ static int mp3_read_header(AVFormatContext *s)
>              if (ret >= 0 &&
>                  (header & SAME_HEADER_MASK) == (header2 & SAME_HEADER_MASK))
>              {
> -                av_log(s, AV_LOG_INFO, "Skipping %d bytes of junk at %"PRId64".\n", i, off);
> +                av_log(s, i > 0 ? AV_LOG_INFO : AV_LOG_VERBOSE, "Skipping %d bytes of junk at %"PRId64".\n", i, off);
>                  ret = avio_seek(s->pb, off + i, SEEK_SET);
>                  if (ret < 0)
>                      return ret;

Applied.