Re: [FFmpeg-devel] [PATCH] avformat/matroskaenc: Accept time base hint

2017-01-14 Thread Michael Bradshaw
On Sat, Jan 14, 2017 at 3:57 AM, Michael Niedermayer wrote: > On Tue, Dec 27, 2016 at 09:47:47PM -0800, Michael Bradshaw wrote: > > +// ms precision is the de-facto standard timescale for mkv files > > +mkv->timecode_scale = 100; > > + > > +// If the user has supplied a desired ti

Re: [FFmpeg-devel] [PATCH] avformat/matroskaenc: Accept time base hint

2017-01-14 Thread Michael Niedermayer
On Fri, Jan 13, 2017 at 08:48:01AM -0800, Michael Bradshaw wrote: > On Wed, Dec 28, 2016 at 5:03 AM, Michael Niedermayer > wrote: > > > On Tue, Dec 27, 2016 at 09:47:47PM -0800, Michael Bradshaw wrote: > > > From: Michael Bradshaw > > > > > > Signed-off-by: Michael Bradshaw > > > --- > > > lib

Re: [FFmpeg-devel] [PATCH] avformat/matroskaenc: Accept time base hint

2017-01-14 Thread Michael Niedermayer
On Tue, Dec 27, 2016 at 09:47:47PM -0800, Michael Bradshaw wrote: > From: Michael Bradshaw > > Signed-off-by: Michael Bradshaw > --- > libavformat/matroskaenc.c | 38 +++--- > 1 file changed, 31 insertions(+), 7 deletions(-) > > diff --git a/libavformat/matroska

Re: [FFmpeg-devel] [PATCH] avformat/matroskaenc: Accept time base hint

2017-01-13 Thread Michael Bradshaw
On Wed, Dec 28, 2016 at 5:03 AM, Michael Niedermayer wrote: > On Tue, Dec 27, 2016 at 09:47:47PM -0800, Michael Bradshaw wrote: > > From: Michael Bradshaw > > > > Signed-off-by: Michael Bradshaw > > --- > > libavformat/matroskaenc.c | 38 +++--- > > 1 file chang

Re: [FFmpeg-devel] [PATCH] avformat/matroskaenc: Accept time base hint

2016-12-28 Thread Michael Niedermayer
On Tue, Dec 27, 2016 at 09:47:47PM -0800, Michael Bradshaw wrote: > From: Michael Bradshaw > > Signed-off-by: Michael Bradshaw > --- > libavformat/matroskaenc.c | 38 +++--- > 1 file changed, 31 insertions(+), 7 deletions(-) breaks fate --- ./tests/ref/fate/bins

[FFmpeg-devel] [PATCH] avformat/matroskaenc: Accept time base hint

2016-12-27 Thread Michael Bradshaw
From: Michael Bradshaw Signed-off-by: Michael Bradshaw --- libavformat/matroskaenc.c | 38 +++--- 1 file changed, 31 insertions(+), 7 deletions(-) diff --git a/libavformat/matroskaenc.c b/libavformat/matroskaenc.c index 827d755..2c2c930 100644 --- a/libavformat/

[FFmpeg-devel] [PATCH] avformat/matroskaenc: Accept time base hint

2016-12-27 Thread Michael Bradshaw
From: Michael Bradshaw Hi, Attached match makes the MKV/WebM muxer accept the time_base hint as specified in AVStream[1]. If the time_base of all the streams are the same and can be represented in Matroska, then the time_base hint is used. Otherwise, it defaults to milliseconds (which is the