diff options
| author | Clément Bœsch <clement.boesch@smartjog.com> | 2012-08-03 10:21:24 +0200 |
|---|---|---|
| committer | Clément Bœsch <ubitux@gmail.com> | 2012-08-08 09:11:25 +0200 |
| commit | aad737c669b9f0d5c58cafce8a8dd97b69ad266b (patch) | |
| tree | b1829405dbaaaa1c02d79acbac0a393a0a9939d3 /libavformat/mxfenc.c | |
| parent | 82e5c5d45bc94fad3bf8bb4de5d73ed30f2d86e3 (diff) | |
lavf/mxfenc: simplify frame rate checks.
Diffstat (limited to 'libavformat/mxfenc.c')
| -rw-r--r-- | libavformat/mxfenc.c | 22 |
1 files changed, 12 insertions, 10 deletions
diff --git a/libavformat/mxfenc.c b/libavformat/mxfenc.c index 6717d778f4..58c31114e5 100644 --- a/libavformat/mxfenc.c +++ b/libavformat/mxfenc.c @@ -1683,26 +1683,28 @@ static int mxf_write_header(AVFormatContext *s) } if (st->codec->codec_type == AVMEDIA_TYPE_VIDEO) { - AVRational rate; + AVRational rate, tbc = st->codec->time_base; // Default component depth to 8 sc->component_depth = 8; - if (fabs(av_q2d(st->codec->time_base) - 1/25.0) < 0.0001) { + mxf->timecode_base = (tbc.den + tbc.num/2) / tbc.num; + switch (mxf->timecode_base) { + case 25: samples_per_frame = PAL_samples_per_frame; mxf->time_base = (AVRational){ 1, 25 }; - mxf->timecode_base = 25; - } else if (fabs(av_q2d(st->codec->time_base) - 1/50.0) < 0.0001) { + break; + case 50: samples_per_frame = PAL_50_samples_per_frame; mxf->time_base = (AVRational){ 1, 50 }; - mxf->timecode_base = 50; - } else if (fabs(av_q2d(st->codec->time_base) - 1001/30000.0) < 0.0001) { + break; + case 30: samples_per_frame = NTSC_samples_per_frame; mxf->time_base = (AVRational){ 1001, 30000 }; - mxf->timecode_base = 30; - } else if (fabs(av_q2d(st->codec->time_base) - 1001/60000.0) < 0.0001) { + break; + case 60: samples_per_frame = NTSC_60_samples_per_frame; mxf->time_base = (AVRational){ 1001, 60000 }; - mxf->timecode_base = 60; - } else { + break; + default: av_log(s, AV_LOG_ERROR, "unsupported video frame rate\n"); return -1; } |