[FFmpeg-devel] [PATCH] lavf/matroskadec: add missing linebreaks in error messages

Andreas Rheinhardt andreas.rheinhardt at outlook.com
Thu Mar 7 02:39:39 EET 2024


Anton Khirnov:
> ---
>  libavformat/matroskadec.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c
> index 5d3d18a146..6d6a60c552 100644
> --- a/libavformat/matroskadec.c
> +++ b/libavformat/matroskadec.c
> @@ -2036,7 +2036,7 @@ static int matroska_parse_content_encodings(MatroskaTrackEncoding *encodings,
>  {
>      if (nb_encodings > 1) {
>          av_log(logctx, AV_LOG_ERROR,
> -                "Multiple combined encodings not supported");
> +                "Multiple combined encodings not supported\n");
>          return 0;
>      }
>      if (!nb_encodings)
> @@ -2055,7 +2055,7 @@ static int matroska_parse_content_encodings(MatroskaTrackEncoding *encodings,
>                               encodings->encryption.key_id.size);
>          } else {
>              encodings->scope = 0;
> -            av_log(logctx, AV_LOG_ERROR, "Unsupported encoding type");
> +            av_log(logctx, AV_LOG_ERROR, "Unsupported encoding type\n");
>          }
>      } else if (
>  #if CONFIG_ZLIB
> @@ -2067,7 +2067,7 @@ static int matroska_parse_content_encodings(MatroskaTrackEncoding *encodings,
>              encodings->compression.algo != MATROSKA_TRACK_ENCODING_COMP_LZO   &&
>              encodings->compression.algo != MATROSKA_TRACK_ENCODING_COMP_HEADERSTRIP) {
>          encodings->scope = 0;
> -        av_log(logctx, AV_LOG_ERROR, "Unsupported encoding type");
> +        av_log(logctx, AV_LOG_ERROR, "Unsupported encoding type\n");
>      } else if (track->codec_priv.size && encodings[0].scope & 2) {
>          uint8_t *codec_priv = track->codec_priv.data;
>          int ret = matroska_decode_buffer(&track->codec_priv.data,

LGTM.

- Andreas



More information about the ffmpeg-devel mailing list