diff mbox series

[FFmpeg-devel] libavformat/matroskaenc: move a warning log to debug log

Message ID 20201106232227.2743493-1-tfoucu@gmail.com
State Withdrawn
Headers show
Series [FFmpeg-devel] libavformat/matroskaenc: move a warning log to debug log
Related show

Checks

Context Check Description
andriy/x86_make success Make finished
andriy/x86_make_fate success Make fate finished
andriy/PPC64_make success Make finished
andriy/PPC64_make_fate success Make fate finished

Commit Message

Thierry Foucu Nov. 6, 2020, 11:22 p.m. UTC
At high frame rate, the message "Starting new cluster due to timestamp"
happens too often. Moving to debug to reduce the warning log
---
 libavformat/matroskaenc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Andreas Rheinhardt Nov. 6, 2020, 11:26 p.m. UTC | #1
Thierry Foucu:
> At high frame rate, the message "Starting new cluster due to timestamp"
> happens too often. Moving to debug to reduce the warning log
> ---
>  libavformat/matroskaenc.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavformat/matroskaenc.c b/libavformat/matroskaenc.c
> index 233c472b8f..97e146479a 100644
> --- a/libavformat/matroskaenc.c
> +++ b/libavformat/matroskaenc.c
> @@ -2292,7 +2292,7 @@ static int mkv_write_packet_internal(AVFormatContext *s, const AVPacket *pkt)
>              ret = mkv_end_cluster(s);
>              if (ret < 0)
>                  return ret;
> -            av_log(s, AV_LOG_WARNING, "Starting new cluster due to timestamp\n");
> +            av_log(s, AV_LOG_DEBUG, "Starting new cluster due to timestamp\n");
>          }
>      }
>  
> 
Can you give more details about this? Getting this warning is actually
an indicator that the packets are not properly interleaved when they
reach the muxer.

- Andreas
diff mbox series

Patch

diff --git a/libavformat/matroskaenc.c b/libavformat/matroskaenc.c
index 233c472b8f..97e146479a 100644
--- a/libavformat/matroskaenc.c
+++ b/libavformat/matroskaenc.c
@@ -2292,7 +2292,7 @@  static int mkv_write_packet_internal(AVFormatContext *s, const AVPacket *pkt)
             ret = mkv_end_cluster(s);
             if (ret < 0)
                 return ret;
-            av_log(s, AV_LOG_WARNING, "Starting new cluster due to timestamp\n");
+            av_log(s, AV_LOG_DEBUG, "Starting new cluster due to timestamp\n");
         }
     }