Re: [FFmpeg-devel] [PATCH] Add coded_width/height to AVCodecParameters

2016-04-03 Thread Derek Buitenhuis
On 4/3/2016 9:49 PM, Clément Bœsch wrote: > dv needed something like this too, but the code was pretty much hacky in > the first place and it was changed not to use them, which avoided such > commit. Where is this needed exactly? Look at the new rv20 test from 1 April. - Derek ___

Re: [FFmpeg-devel] [PATCH] Add coded_width/height to AVCodecParameters

2016-04-03 Thread Clément Bœsch
On Sun, Apr 03, 2016 at 02:07:47PM +0200, Michael Niedermayer wrote: > Without this or some other change moving data back and forth corrupts it > as the cropped & possibly low res resolution modified w/h will be taken > a the pre lowres resolution > also without 2 sets theres no place to communicat

Re: [FFmpeg-devel] [PATCH] Add coded_width/height to AVCodecParameters

2016-04-03 Thread Michael Niedermayer
On Sun, Apr 03, 2016 at 03:31:59PM +0200, wm4 wrote: > On Sun, 3 Apr 2016 15:26:02 +0200 > Michael Niedermayer wrote: > > > On Sun, Apr 03, 2016 at 02:31:06PM +0200, Hendrik Leppkes wrote: > > > On Sun, Apr 3, 2016 at 2:07 PM, Michael Niedermayer > > > wrote: > > > > Without this or some other

Re: [FFmpeg-devel] [PATCH] Add coded_width/height to AVCodecParameters

2016-04-03 Thread wm4
On Sun, 3 Apr 2016 15:26:02 +0200 Michael Niedermayer wrote: > On Sun, Apr 03, 2016 at 02:31:06PM +0200, Hendrik Leppkes wrote: > > On Sun, Apr 3, 2016 at 2:07 PM, Michael Niedermayer > > wrote: > > > Without this or some other change moving data back and forth corrupts it > > > as the cropped

Re: [FFmpeg-devel] [PATCH] Add coded_width/height to AVCodecParameters

2016-04-03 Thread Michael Niedermayer
On Sun, Apr 03, 2016 at 02:31:06PM +0200, Hendrik Leppkes wrote: > On Sun, Apr 3, 2016 at 2:07 PM, Michael Niedermayer > wrote: > > Without this or some other change moving data back and forth corrupts it > > as the cropped & possibly low res resolution modified w/h will be taken > > a the pre low

Re: [FFmpeg-devel] [PATCH] Add coded_width/height to AVCodecParameters

2016-04-03 Thread wm4
On Sun, 3 Apr 2016 14:31:06 +0200 Hendrik Leppkes wrote: > On Sun, Apr 3, 2016 at 2:07 PM, Michael Niedermayer > wrote: > > Without this or some other change moving data back and forth corrupts it > > as the cropped & possibly low res resolution modified w/h will be taken > > a the pre lowres re

Re: [FFmpeg-devel] [PATCH] Add coded_width/height to AVCodecParameters

2016-04-03 Thread Hendrik Leppkes
On Sun, Apr 3, 2016 at 2:07 PM, Michael Niedermayer wrote: > Without this or some other change moving data back and forth corrupts it > as the cropped & possibly low res resolution modified w/h will be taken > a the pre lowres resolution > also without 2 sets theres no place to communicate the cro

[FFmpeg-devel] [PATCH] Add coded_width/height to AVCodecParameters

2016-04-03 Thread Michael Niedermayer
Without this or some other change moving data back and forth corrupts it as the cropped & possibly low res resolution modified w/h will be taken a the pre lowres resolution also without 2 sets theres no place to communicate the cropped dimensions can be reproduced with ./ffplay_g -vlowres 2 ticket