[FFmpeg-devel] libavcodec/h264_sei: Don't log random user data.

[Michael Niedermayer]

On Fri, May 05, 2017 at 09:30:54PM +0000, Kieran Kunhya wrote:
> From d43c0366d4ba128772dc7909152f4e9635f269cb Mon Sep 17 00:00:00 2001
> From: Kieran Kunhya <kierank at obe.tv>
> Date: Fri, 5 May 2017 14:29:59 -0700
> Subject: [PATCH] libavcodec/h264_sei: Don't log random user data. This
>  prevents terminal junk.
> 
> ---
>  libavcodec/h264_sei.c | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/libavcodec/h264_sei.c b/libavcodec/h264_sei.c
> index a7e627eba3..c81ddfef4f 100644
> --- a/libavcodec/h264_sei.c
> +++ b/libavcodec/h264_sei.c
> @@ -257,9 +257,6 @@ static int
> decode_unregistered_user_data(H264SEIUnregistered *h, GetBitContext *
>      if (e == 1 && build == 1 && !strncmp(user_data+16, "x264 - core 0000",
> 16))
>          h->x264_build = 67;
> 
> -    if (strlen(user_data + 16) > 0)
> -        av_log(logctx, AV_LOG_DEBUG, "user data:\"%s\"\n", user_data + 16);

This is just at debug level.
Do you have an example (file/usecase) where this causes excessive
output ?


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Freedom in capitalist society always remains about the same as it was in
ancient Greek republics: Freedom for slave owners. -- Vladimir Lenin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170506/c0d33af0/attachment.sig>