Message ID | CAB0OVGoNJu6xCV4hUiwyy2fWC46RwcGOEbEWdRzuXMFO6xV4uw@mail.gmail.com |
---|---|
State | Accepted |
Headers | show |
2017-10-31 0:22 GMT+01:00 Carl Eugen Hoyos <ceffmpeg@gmail.com>: > Hi! > > Kasper Folman has reported that FFmpeg does not write 60fps > mxf output and has provided a 60fps mxf file on -users. > > Attached patch allows 60fps output. This was successfully tested so I will push if there are no objections. Carl Eugen
On 11/16/2017 1:27 PM, Carl Eugen Hoyos wrote:
> This was successfully tested so I will push if there are no objections.
Seems harmless enough if it isn't violating some spec (it is MXF after all...)
If someone had objections, they would have responded by now, probably.
- Derek
2017-11-16 15:08 GMT+01:00 Derek Buitenhuis <derek.buitenhuis@gmail.com>: > On 11/16/2017 1:27 PM, Carl Eugen Hoyos wrote: >> This was successfully tested so I will push if there are no objections. > > Seems harmless enough if it isn't violating some spec (it is MXF after all...) > > If someone had objections, they would have responded by now, probably. Patch applied. Carl Eugen
From bb5b071db9a8977bd1a7dea579b4d244e17ca3a3 Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos <ceffmpeg@gmail.com> Date: Tue, 31 Oct 2017 00:19:38 +0100 Subject: [PATCH] lavf/mxfenc: Support 60fps output. --- libavformat/mxf.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavformat/mxf.c b/libavformat/mxf.c index bfc3218..5994b09 100644 --- a/libavformat/mxf.c +++ b/libavformat/mxf.c @@ -137,6 +137,7 @@ static const MXFSamplesPerFrame mxf_spf[] = { { { 1001, 60000 }, { 801, 801, 801, 801, 800, 0 } }, // NTSC 59.94 { { 1, 25 }, { 1920, 0, 0, 0, 0, 0 } }, // PAL 25 { { 1, 50 }, { 960, 0, 0, 0, 0, 0 } }, // PAL 50 + { { 1, 60 }, { 800, 0, 0, 0, 0, 0 } }, }; static const AVRational mxf_time_base[] = { @@ -146,6 +147,7 @@ static const AVRational mxf_time_base[] = { { 1001, 60000 }, { 1, 25 }, { 1, 50 }, + { 1, 60 }, { 0, 0} }; -- 1.7.10.4