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
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel