[FFmpeg-devel] [PATCH 02/11] ffprobe: use av_mpegtc_to_timecode_string().

Stefano Sabatini stefasab at gmail.com
Mon Jan 16 19:01:46 CET 2012


On date Monday 2012-01-16 17:30:05 +0100, Clément Bœsch encoded:
> From: Clément Bœsch <clement.boesch at smartjog.com>
> 
> ---
>  ffprobe.c |   11 ++++-------
>  1 files changed, 4 insertions(+), 7 deletions(-)
> 
> diff --git a/ffprobe.c b/ffprobe.c
> index 61c66ae..55cf43a 100644
> --- a/ffprobe.c
> +++ b/ffprobe.c
> @@ -32,6 +32,7 @@
>  #include "libavutil/opt.h"
>  #include "libavutil/pixdesc.h"
>  #include "libavutil/dict.h"
> +#include "libavutil/timecode.h"
>  #include "libavdevice/avdevice.h"
>  #include "libswscale/swscale.h"
>  #include "libswresample/swresample.h"
> @@ -1454,13 +1455,9 @@ static void show_stream(WriterContext *w, AVFormatContext *fmt_ctx, int stream_i
>              else   print_str_opt("pix_fmt", "unknown");
>              print_int("level",   dec_ctx->level);
>              if (dec_ctx->timecode_frame_start >= 0) {
> -                uint32_t tc = dec_ctx->timecode_frame_start;
> -                print_fmt("timecode", "%02d:%02d:%02d%c%02d",
> -                          tc>>19 & 0x1f,              // hours
> -                          tc>>13 & 0x3f,              // minutes
> -                          tc>>6  & 0x3f,              // seconds
> -                          tc     & 1<<24 ? ';' : ':', // drop
> -                          tc     & 0x3f);             // frames

> +                char tcbuf[AVTIMECODE_STR_LEN];

Unrelated nit+bikeshed: don't know if this is already public, but
maybe the AV_* convenction should be favored over AV* (I mean
AVTIMECODE.. -> AV_TIMECODE...).

> +                av_mpegtc_to_timecode_string(tcbuf, dec_ctx->timecode_frame_start);
> +                print_str("timecode", tcbuf);

Looks fine otherwise, thanks.
-- 
FFmpeg = Fast and Friendly Martial Perennial Exuberant Genius


More information about the ffmpeg-devel mailing list