Re: [FFmpeg-devel] [PATCH]lavf/mxfdec: Set width to actual coded_width for AVCI50

2015-11-29 Thread Carl Eugen Hoyos
On Friday 27 November 2015 09:46:48 pm Tomas Härdin wrote: > > Ok to apply? > > Sure, we can take the renaming separately. I applied the patch. Thank you, Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinf

Re: [FFmpeg-devel] [PATCH]lavf/mxfdec: Set width to actual coded_width for AVCI50

2015-11-27 Thread Tomas Härdin
On Thu, 2015-11-26 at 00:48 +0100, Carl Eugen Hoyos wrote: > On Wednesday 25 November 2015 12:57:07 am Tomas Härdin wrote: > > On Tue, 2015-11-24 at 12:02 +0100, Carl Eugen Hoyos wrote: > > > Hi! > > > > > > Attached patch fixes ticket #5029. > > > > > > Please comment, Carl Eugen > > > > Looks sim

Re: [FFmpeg-devel] [PATCH]lavf/mxfdec: Set width to actual coded_width for AVCI50

2015-11-25 Thread Carl Eugen Hoyos
On Wednesday 25 November 2015 12:57:07 am Tomas Härdin wrote: > On Tue, 2015-11-24 at 12:02 +0100, Carl Eugen Hoyos wrote: > > Hi! > > > > Attached patch fixes ticket #5029. > > > > Please comment, Carl Eugen > > Looks simple enough, but (ab)using MXFCodecUL like that > has a slight stink to it.

Re: [FFmpeg-devel] [PATCH]lavf/mxfdec: Set width to actual coded_width for AVCI50

2015-11-24 Thread Tomas Härdin
On Tue, 2015-11-24 at 12:02 +0100, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes ticket #5029. > > Please comment, Carl Eugen Looks simple enough, but (ab)using MXFCodecUL like that has a slight stink to it. Can we perhaps rename the struct to something that signals its more general use