[FFmpeg-devel] [PATCH 2/4] lavc: include timebase in avcodec string at debug level.

[Paul B Mahol]

On 1/17/13, Michael Niedermayer <michaelni at gmx.at> wrote:
> Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> ---
>  libavcodec/utils.c |    6 ++++++
>  1 file changed, 6 insertions(+)
>
> diff --git a/libavcodec/utils.c b/libavcodec/utils.c
> index 8c0aa98..165b083 100644
> --- a/libavcodec/utils.c
> +++ b/libavcodec/utils.c
> @@ -2121,6 +2121,12 @@ void avcodec_string(char *buf, int buf_size,
> AVCodecContext *enc, int encode)
>          }
>          break;
>      default:

really default?

> +        if (av_log_get_level() >= AV_LOG_DEBUG) {
> +            int g = av_gcd(enc->time_base.num, enc->time_base.den);

Could this return 0 for g?

> +            snprintf(buf + strlen(buf), buf_size - strlen(buf),
> +                        ", %d/%d",
> +                        enc->time_base.num / g, enc->time_base.den / g);
> +        }


>          return;
>      }
>      if (encode) {
> --
> 1.7.9.5
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>