Re: [FFmpeg-devel] [PATCH 2/2] lavf/mxfdec: begin utilizing the newly parsed widths and heights

2016-09-27 Thread Marton Balint
On Mon, 26 Sep 2016, Jan Ekstrom wrote: On Sep 26, 2016 04:05, "Marton Balint" wrote: Overriding width/height with display width/height does not seem right, check what happens with a PAL IMX50 MXF file for example. If you want to signal this, then a stream side data with

Re: [FFmpeg-devel] [PATCH 2/2] lavf/mxfdec: begin utilizing the newly parsed widths and heights

2016-09-26 Thread Jan Ekstrom
On Sep 26, 2016 04:05, "Marton Balint" wrote: > > Overriding width/height with display width/height does not seem right, check > what happens with a PAL IMX50 MXF file for example. If you want to signal > this, then a stream side data with some AVPanScan values might make more

Re: [FFmpeg-devel] [PATCH 2/2] lavf/mxfdec: begin utilizing the newly parsed widths and heights

2016-09-25 Thread Marton Balint
On Mon, 26 Sep 2016, Jan Ekström wrote: * Updates the width/height sanity check to check all values against INT_MAX. * Correctly utilizes the stored width or height by default, and if sampled or display values are available they are utilized. Signed-off-by: Jan Ekström