[FFmpeg-devel] [PATCH]Show subtitle resolution in avcodec_string()

[Nicolas George]

Le septidi 17 fructidor, an CCXXI, Carl Eugen Hoyos a écrit :
> Hi!
> 
> Attached patch shows the user the problem with files like from ticket #679 or 
> #2930.
> 
> Please comment, Carl Eugen

> diff --git a/libavcodec/utils.c b/libavcodec/utils.c
> index 4e510e6..497cf15 100644
> --- a/libavcodec/utils.c
> +++ b/libavcodec/utils.c
> @@ -2668,6 +2668,11 @@ void avcodec_string(char *buf, int buf_size, AVCodecContext *enc, int encode)
>                           enc->time_base.num / g, enc->time_base.den / g);
>          }
>          break;
> +    case AVMEDIA_TYPE_SUBTITLE:
> +        if (enc->width)
> +            snprintf(buf + strlen(buf), buf_size - strlen(buf),
> +                     ", %dx%d", enc->width, enc->height);
> +        break;
>      default:
>          return;
>      }

I think this is a good idea, and I see no problem with the patch. But I do
not maintain that file.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130903/a2a62f3f/attachment.asc>