[FFmpeg-devel] [PATCH] Move packed B-frames message level to info.

Alexander Strasser eclipse7 at gmx.net
Sun Sep 8 23:34:31 CEST 2013


On 2013-09-08 19:49 +0200, Reimar Döffinger wrote:
> Since they generally decode fine, printing it as a warning
> might not really be appropriate.
> 
> Signed-off-by: Reimar Döffinger <Reimar.Doeffinger at gmx.de>
> ---
>  libavcodec/mpeg4videodec.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavcodec/mpeg4videodec.c b/libavcodec/mpeg4videodec.c
> index c81a917..fa32aa6 100644
> --- a/libavcodec/mpeg4videodec.c
> +++ b/libavcodec/mpeg4videodec.c
> @@ -1912,7 +1912,7 @@ static int decode_user_data(MpegEncContext *s, GetBitContext *gb){
>          s->divx_build= build;
>          s->divx_packed= e==3 && last=='p';
>          if(s->divx_packed && !s->showed_packed_warning) {
> -            av_log(s->avctx, AV_LOG_WARNING, "Video uses a non-standard and "
> +            av_log(s->avctx, AV_LOG_INFO, "Video uses a non-standard and "
>                     "wasteful way to store B-frames ('packed B-frames'). "
>                     "Consider using a tool like VirtualDub or avidemux to fix it.\n");
>              s->showed_packed_warning=1;

  This and the previous patch seem fine to me. (Though I am not
maintaining this file.)

  Alexander
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130908/11bce109/attachment.asc>


More information about the ffmpeg-devel mailing list