Re: [FFmpeg-devel] [FFmpeg-cvslog] avutil/imgutils: remove special case for aligning the palette

2016-02-22 Thread wm4
On Sat, 20 Feb 2016 21:38:30 + Derek Buitenhuis wrote: > On 2/20/2016 9:24 PM, Michael Niedermayer wrote: > >> This is a silent API break. You changed behavior of a function in such a > >> way > >> that functioning code no longer works. > > > > yes, i posted a

Re: [FFmpeg-devel] [FFmpeg-cvslog] avutil/imgutils: remove special case for aligning the palette

2016-02-20 Thread Michael Niedermayer
On Sat, Feb 20, 2016 at 05:41:40PM +, Derek Buitenhuis wrote: > > ffmpeg | branch: master | Michael Niedermayer | > > Sun Feb 14 15:22:47 2016 +0100| [0eb4092c1bf4d74c3b9a65fb0dbe4f90d3a316d0] > > | committer: Michael Niedermayer > > [...] > > > +if (align < 4) { > > +

Re: [FFmpeg-devel] [FFmpeg-cvslog] avutil/imgutils: remove special case for aligning the palette

2016-02-20 Thread wm4
On Sat, 20 Feb 2016 17:41:40 + Derek Buitenhuis wrote: > > ffmpeg | branch: master | Michael Niedermayer | > > Sun Feb 14 15:22:47 2016 +0100| [0eb4092c1bf4d74c3b9a65fb0dbe4f90d3a316d0] > > | committer: Michael Niedermayer > > [...] > > > +if (align

Re: [FFmpeg-devel] [FFmpeg-cvslog] avutil/imgutils: remove special case for aligning the palette

2016-02-20 Thread Derek Buitenhuis
> ffmpeg | branch: master | Michael Niedermayer | > Sun Feb 14 15:22:47 2016 +0100| [0eb4092c1bf4d74c3b9a65fb0dbe4f90d3a316d0] | > committer: Michael Niedermayer [...] > +if (align < 4) { > +av_log(NULL, AV_LOG_ERROR, "Formats with a palette require a > minimum alignment