[FFmpeg-devel] [libav-devel] [PATCH 1/2] mxfenc: fix ignored drop flag in binary timecode representation.

Ronald S. Bultje rsbultje at gmail.com
Tue Jul 5 17:04:00 CEST 2011


Hi,

2011/7/4 Clément Bœsch <ubitux at gmail.com>:
> From: Clément Bœsch <clement.boesch at smartjog.com>
>
> ---
>  libavformat/mxfenc.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/libavformat/mxfenc.c b/libavformat/mxfenc.c
> index c6532a3..0f230cb 100644
> --- a/libavformat/mxfenc.c
> +++ b/libavformat/mxfenc.c
> @@ -1539,7 +1539,7 @@ static const uint8_t system_metadata_package_set_key[] = { 0x06,0x0E,0x2B,0x34,0
>  static uint32_t ff_framenum_to_12m_time_code(unsigned frame, int drop, int fps)
>  {
>     return (0                                    << 31) | // color frame flag
> -           (0                                    << 30) | // drop  frame flag
> +           (drop                                 << 30) | // drop  frame flag
>            ( ((frame % fps) / 10)                << 28) | // tens  of frames
>            ( ((frame % fps) % 10)                << 24) | // units of frames
>            (0                                    << 23) | // field phase (NTSC), b0 (PAL)

I'm certainly no MXF expert, but this is probably OK.

Ronald


More information about the ffmpeg-devel mailing list