Re: [FFmpeg-devel] [PATCH] ffmpeg: drop format specific stream copy heuristics

2016-09-08 Thread Michael Niedermayer
On Wed, Sep 07, 2016 at 10:07:26PM -0300, James Almer wrote: > On 9/7/2016 6:14 AM, Michael Niedermayer wrote: > >> libavformat/utils.c |4 +++- > >> > tests/ref/fate/copy-trac4914 |4 ++-- > >> > tests/ref/fate/copy-trac4914-avi |4 ++-- > >> > 3 files changed, 7 inse

Re: [FFmpeg-devel] [PATCH] ffmpeg: drop format specific stream copy heuristics

2016-09-08 Thread Clément Bœsch
On Wed, Sep 07, 2016 at 10:07:26PM -0300, James Almer wrote: > On 9/7/2016 6:14 AM, Michael Niedermayer wrote: > >> libavformat/utils.c |4 +++- > >> > tests/ref/fate/copy-trac4914 |4 ++-- > >> > tests/ref/fate/copy-trac4914-avi |4 ++-- > >> > 3 files changed, 7 inse

Re: [FFmpeg-devel] [PATCH] ffmpeg: drop format specific stream copy heuristics

2016-09-07 Thread James Almer
On 9/7/2016 6:14 AM, Michael Niedermayer wrote: >> libavformat/utils.c |4 +++- >> > tests/ref/fate/copy-trac4914 |4 ++-- >> > tests/ref/fate/copy-trac4914-avi |4 ++-- >> > 3 files changed, 7 insertions(+), 5 deletions(-) >> > 2dc146e807cbdbdbca653a22d827920e8c05b3c8

Re: [FFmpeg-devel] [PATCH] ffmpeg: drop format specific stream copy heuristics

2016-09-07 Thread Michael Niedermayer
On Tue, Sep 06, 2016 at 06:26:59PM +0200, Michael Niedermayer wrote: > On Tue, Sep 06, 2016 at 05:45:46PM +0200, Hendrik Leppkes wrote: > > On Tue, Sep 6, 2016 at 5:10 PM, Michael Niedermayer > > wrote: > > > On Tue, Sep 06, 2016 at 02:39:11PM +0200, Hendrik Leppkes wrote: > > >> On Tue, Sep 6, 20

Re: [FFmpeg-devel] [PATCH] ffmpeg: drop format specific stream copy heuristics

2016-09-06 Thread Michael Niedermayer
On Tue, Sep 06, 2016 at 05:45:46PM +0200, Hendrik Leppkes wrote: > On Tue, Sep 6, 2016 at 5:10 PM, Michael Niedermayer > wrote: > > On Tue, Sep 06, 2016 at 02:39:11PM +0200, Hendrik Leppkes wrote: > >> On Tue, Sep 6, 2016 at 2:21 PM, Michael Niedermayer > >> wrote: > >> > On Tue, Sep 06, 2016 at

Re: [FFmpeg-devel] [PATCH] ffmpeg: drop format specific stream copy heuristics

2016-09-06 Thread Hendrik Leppkes
On Tue, Sep 6, 2016 at 5:10 PM, Michael Niedermayer wrote: > On Tue, Sep 06, 2016 at 02:39:11PM +0200, Hendrik Leppkes wrote: >> On Tue, Sep 6, 2016 at 2:21 PM, Michael Niedermayer >> wrote: >> > On Tue, Sep 06, 2016 at 02:18:35PM +0200, Michael Niedermayer wrote: >> >> On Tue, Sep 06, 2016 at 04

Re: [FFmpeg-devel] [PATCH] ffmpeg: drop format specific stream copy heuristics

2016-09-06 Thread Michael Niedermayer
On Tue, Sep 06, 2016 at 11:32:24AM -0300, James Almer wrote: > On 9/6/2016 9:57 AM, Clément Bœsch wrote: > >> We don't have time_base in codecpar, so why do we need ticks per frame in > >> it? > >> > > >> > Which time_base does it modify the interpretation of? The field should > >> > be bundled w

Re: [FFmpeg-devel] [PATCH] ffmpeg: drop format specific stream copy heuristics

2016-09-06 Thread Michael Niedermayer
On Tue, Sep 06, 2016 at 02:39:11PM +0200, Hendrik Leppkes wrote: > On Tue, Sep 6, 2016 at 2:21 PM, Michael Niedermayer > wrote: > > On Tue, Sep 06, 2016 at 02:18:35PM +0200, Michael Niedermayer wrote: > >> On Tue, Sep 06, 2016 at 04:57:06AM +0200, Michael Niedermayer wrote: > >> > On Mon, Sep 05,

Re: [FFmpeg-devel] [PATCH] ffmpeg: drop format specific stream copy heuristics

2016-09-06 Thread Clément Bœsch
On Tue, Sep 06, 2016 at 11:32:24AM -0300, James Almer wrote: > On 9/6/2016 9:57 AM, Clément Bœsch wrote: > >> We don't have time_base in codecpar, so why do we need ticks per frame in > >> it? > >> > > >> > Which time_base does it modify the interpretation of? The field should > >> > be bundled w

Re: [FFmpeg-devel] [PATCH] ffmpeg: drop format specific stream copy heuristics

2016-09-06 Thread James Almer
On 9/6/2016 9:57 AM, Clément Bœsch wrote: >> We don't have time_base in codecpar, so why do we need ticks per frame in it? >> > >> > Which time_base does it modify the interpretation of? The field should >> > be bundled with that, then. > When do we have a mismatch of st->time_base and that "codec

Re: [FFmpeg-devel] [PATCH] ffmpeg: drop format specific stream copy heuristics

2016-09-06 Thread Clément Bœsch
On Tue, Sep 06, 2016 at 02:39:11PM +0200, Hendrik Leppkes wrote: > On Tue, Sep 6, 2016 at 2:21 PM, Michael Niedermayer > wrote: > > On Tue, Sep 06, 2016 at 02:18:35PM +0200, Michael Niedermayer wrote: > >> On Tue, Sep 06, 2016 at 04:57:06AM +0200, Michael Niedermayer wrote: > >> > On Mon, Sep 05,

Re: [FFmpeg-devel] [PATCH] ffmpeg: drop format specific stream copy heuristics

2016-09-06 Thread Hendrik Leppkes
On Tue, Sep 6, 2016 at 2:21 PM, Michael Niedermayer wrote: > On Tue, Sep 06, 2016 at 02:18:35PM +0200, Michael Niedermayer wrote: >> On Tue, Sep 06, 2016 at 04:57:06AM +0200, Michael Niedermayer wrote: >> > On Mon, Sep 05, 2016 at 10:04:35PM -0300, James Almer wrote: >> > > On 9/5/2016 12:41 PM, M

Re: [FFmpeg-devel] [PATCH] ffmpeg: drop format specific stream copy heuristics

2016-09-06 Thread Michael Niedermayer
On Tue, Sep 06, 2016 at 02:18:35PM +0200, Michael Niedermayer wrote: > On Tue, Sep 06, 2016 at 04:57:06AM +0200, Michael Niedermayer wrote: > > On Mon, Sep 05, 2016 at 10:04:35PM -0300, James Almer wrote: > > > On 9/5/2016 12:41 PM, Michael Niedermayer wrote: > > > > On Mon, Sep 05, 2016 at 04:41:5

Re: [FFmpeg-devel] [PATCH] ffmpeg: drop format specific stream copy heuristics

2016-09-06 Thread Michael Niedermayer
On Tue, Sep 06, 2016 at 04:57:06AM +0200, Michael Niedermayer wrote: > On Mon, Sep 05, 2016 at 10:04:35PM -0300, James Almer wrote: > > On 9/5/2016 12:41 PM, Michael Niedermayer wrote: > > > On Mon, Sep 05, 2016 at 04:41:52PM +0200, Clément Bœsch wrote: > > >> From: Clément Bœsch > > >> > > >> The

Re: [FFmpeg-devel] [PATCH] ffmpeg: drop format specific stream copy heuristics

2016-09-05 Thread Michael Niedermayer
On Mon, Sep 05, 2016 at 10:04:35PM -0300, James Almer wrote: > On 9/5/2016 12:41 PM, Michael Niedermayer wrote: > > On Mon, Sep 05, 2016 at 04:41:52PM +0200, Clément Bœsch wrote: > >> From: Clément Bœsch > >> > >> These adjusted codec fields do not seem to be in use anymore and prevent > >> the co

Re: [FFmpeg-devel] [PATCH] ffmpeg: drop format specific stream copy heuristics

2016-09-05 Thread James Almer
On 9/5/2016 10:04 PM, James Almer wrote: > On 9/5/2016 12:41 PM, Michael Niedermayer wrote: >> On Mon, Sep 05, 2016 at 04:41:52PM +0200, Clément Bœsch wrote: >>> From: Clément Bœsch >>> >>> These adjusted codec fields do not seem to be in use anymore and prevent >>> the convert of ffmpeg*.c to cod

Re: [FFmpeg-devel] [PATCH] ffmpeg: drop format specific stream copy heuristics

2016-09-05 Thread James Almer
On 9/5/2016 12:41 PM, Michael Niedermayer wrote: > On Mon, Sep 05, 2016 at 04:41:52PM +0200, Clément Bœsch wrote: >> From: Clément Bœsch >> >> These adjusted codec fields do not seem to be in use anymore and prevent >> the convert of ffmpeg*.c to codecpar. > > ./ffmpeg -i ~/tickets/4914/xdcam8m

Re: [FFmpeg-devel] [PATCH] ffmpeg: drop format specific stream copy heuristics

2016-09-05 Thread Michael Niedermayer
On Mon, Sep 05, 2016 at 05:49:10PM +0200, Clément Bœsch wrote: > On Mon, Sep 05, 2016 at 05:41:25PM +0200, Michael Niedermayer wrote: > > On Mon, Sep 05, 2016 at 04:41:52PM +0200, Clément Bœsch wrote: > > > From: Clément Bœsch > > > > > > These adjusted codec fields do not seem to be in use anymo

Re: [FFmpeg-devel] [PATCH] ffmpeg: drop format specific stream copy heuristics

2016-09-05 Thread Clément Bœsch
On Mon, Sep 05, 2016 at 05:41:25PM +0200, Michael Niedermayer wrote: > On Mon, Sep 05, 2016 at 04:41:52PM +0200, Clément Bœsch wrote: > > From: Clément Bœsch > > > > These adjusted codec fields do not seem to be in use anymore and prevent > > the convert of ffmpeg*.c to codecpar. > > ./ffmpeg

Re: [FFmpeg-devel] [PATCH] ffmpeg: drop format specific stream copy heuristics

2016-09-05 Thread Michael Niedermayer
On Mon, Sep 05, 2016 at 04:41:52PM +0200, Clément Bœsch wrote: > From: Clément Bœsch > > These adjusted codec fields do not seem to be in use anymore and prevent > the convert of ffmpeg*.c to codecpar. ./ffmpeg -i ~/tickets/4914/xdcam8mp2-1s_small.ts -c:v copy out.mxf fails, no output anymore

[FFmpeg-devel] [PATCH] ffmpeg: drop format specific stream copy heuristics

2016-09-05 Thread Clément Bœsch
From: Clément Bœsch These adjusted codec fields do not seem to be in use anymore and prevent the convert of ffmpeg*.c to codecpar. --- ffmpeg.c | 42 -- 1 file changed, 42 deletions(-) diff --git a/ffmpeg.c b/ffmpeg.c index d858407..9516b2d 100644 --- a/f