Re: [FFmpeg-devel] [PATCH 2/4] dpxenc: enforce alignment requirement

2014-08-14 Thread Michael Niedermayer
On Thu, Aug 14, 2014 at 09:36:53AM +0200, Christophe Gisquet wrote: > Hi, > > 2014-08-14 4:30 GMT+02:00 Michael Niedermayer : > >> probably ok > > > > applied > > Now that I think of it, this change causes older versions of ffmpeg to > be unable to decode the output of the fixed encoder. So: > -

Re: [FFmpeg-devel] [PATCH 2/4] dpxenc: enforce alignment requirement

2014-08-14 Thread Christophe Gisquet
Hi, 2014-08-14 4:30 GMT+02:00 Michael Niedermayer : >> probably ok > > applied Now that I think of it, this change causes older versions of ffmpeg to be unable to decode the output of the fixed encoder. So: - Should we bump eg a minor version number to let any decoder have the possibility to dete

Re: [FFmpeg-devel] [PATCH 2/4] dpxenc: enforce alignment requirement

2014-08-13 Thread Michael Niedermayer
On Wed, Aug 13, 2014 at 01:51:42PM +0200, Paul B Mahol wrote: > On 8/13/14, Christophe Gisquet wrote: > > S268M-2003 specifies that each line start is aligned on a 4-byte boundary. > > --- > > libavcodec/dpxenc.c | 44 +--- > > 1 file changed, 37 insertions

Re: [FFmpeg-devel] [PATCH 2/4] dpxenc: enforce alignment requirement

2014-08-13 Thread Paul B Mahol
On 8/13/14, Christophe Gisquet wrote: > S268M-2003 specifies that each line start is aligned on a 4-byte boundary. > --- > libavcodec/dpxenc.c | 44 +--- > 1 file changed, 37 insertions(+), 7 deletions(-) > probably ok _

[FFmpeg-devel] [PATCH 2/4] dpxenc: enforce alignment requirement

2014-08-13 Thread Christophe Gisquet
S268M-2003 specifies that each line start is aligned on a 4-byte boundary. --- libavcodec/dpxenc.c | 44 +--- 1 file changed, 37 insertions(+), 7 deletions(-) diff --git a/libavcodec/dpxenc.c b/libavcodec/dpxenc.c index 0eb1297..059d8c6 100644 --- a/libavco