On Mon, Oct 12, 2015 at 10:39:23PM +0200, Christophe Gisquet wrote:
> Le 12 oct. 2015 21:37, "Michael Niedermayer" <mich...@niedermayer.cc> a
> écrit :
> >
> > On Mon, Oct 12, 2015 at 07:37:43PM +0200, Christophe Gisquet wrote:
> > > It was useful to (accidentally?) spot an overflow in the column pass
> > > of the x86 simple_idct10 implementation.
> > > ---
> > >  tests/fate/vcodec.mak                          | 6 +++++-
> > >  tests/ref/vsynth/vsynth1-dnxhd-1080i-10bit     | 4 ++++
> > >  tests/ref/vsynth/vsynth2-dnxhd-1080i-10bit     | 4 ++++
> > >  tests/ref/vsynth/vsynth3-dnxhd-1080i-10bit     | 4 ++++
> > >  tests/ref/vsynth/vsynth_lena-dnxhd-1080i-10bit | 4 ++++
> > >  5 files changed, 21 insertions(+), 1 deletion(-)
> > >  create mode 100644 tests/ref/vsynth/vsynth1-dnxhd-1080i-10bit
> > >  create mode 100644 tests/ref/vsynth/vsynth2-dnxhd-1080i-10bit
> > >  create mode 100644 tests/ref/vsynth/vsynth3-dnxhd-1080i-10bit
> > >  create mode 100644 tests/ref/vsynth/vsynth_lena-dnxhd-1080i-10bit
> >
> > this alone and with patch 1 doesnt pass fate:
> 
> I'm so bad at this codec stuff.

no, you are not, this stuff is rather convoluted and the mpegvideo
*dct stuff is not well documented.

I think the patchset is mostly fine, ill retest things and will try to
fix the remaining issues (if iam able to) and if successfull ill apply
it i think this one patch here is fine as is if its applied later in
the patch series

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

During times of universal deceit, telling the truth becomes a
revolutionary act. -- George Orwell

Attachment: signature.asc
Description: Digital signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to