Re: [FFmpeg-devel] [PATCH] fate: add test for -time_base option

2017-06-12 Thread Michael Bradshaw
On Sat, Jun 10, 2017 at 8:19 AM, James Almer wrote: > Is mxf as output needed for this? mxf was one of the (relatively few) number of muxers I knew of that would utilize the provided time base. It's not strictly needed. > If not, the framemd5() or framecrc() > fate functions (which use the muxer

Re: [FFmpeg-devel] [PATCH] fate: add test for -time_base option

2017-06-10 Thread James Almer
On 6/9/2017 7:10 PM, Michael Bradshaw wrote: > Hi, > > Attached patch adds a FATE test for the -time_base option, as > promised[1] (months ago). This is my first FATE patch, so please be > sure to check for stupid mistakes and whatnot. I'm happy to revise it > as needed. > > Thanks, > > --Michae

Re: [FFmpeg-devel] [PATCH] fate: add test for -time_base option

2017-06-10 Thread Michael Niedermayer
On Fri, Jun 09, 2017 at 03:10:56PM -0700, Michael Bradshaw wrote: > Hi, > > Attached patch adds a FATE test for the -time_base option, as > promised[1] (months ago). This is my first FATE patch, so please be > sure to check for stupid mistakes and whatnot. I'm happy to revise it > as needed. > >

[FFmpeg-devel] [PATCH] fate: add test for -time_base option

2017-06-09 Thread Michael Bradshaw
Hi, Attached patch adds a FATE test for the -time_base option, as promised[1] (months ago). This is my first FATE patch, so please be sure to check for stupid mistakes and whatnot. I'm happy to revise it as needed. Thanks, --Michael [1]: http://ffmpeg.org/pipermail/ffmpeg-devel/2017-March/20943