[Ffmpeg-devel] [PATCH] Indeo 2 decoder

[Michael Niedermayer]

Hi

On Saturday 09 July 2005 21:04, Diego Biurrun wrote:
> On Thu, Jun 23, 2005 at 08:43:10AM +0200, Michael Niedermayer wrote:
> > On Thursday 23 June 2005 04:49, Kostya wrote:
> > > Here is the patch suggested by unkaggregate(@users.sf.net).
> > > Main reason is: deltas in interframes need scaling by 3/4 before
> > > applying. Detailed description is at:
> > > http://sourceforge.net/tracker/index.php?func=detail&aid=1222099&group_
> > >id=1 6082&atid=116082 He also mentioned some samples at:
> > > http://www.nerdgrounds.com/indeo21_test/
> >
> > i still see some ghosting artefacts in fish05.avi with this patch and
> > frame stepping with 's' in ffplay allthough i didnt try it with the dll
> >
> > could maybe the 3/4 actually be a variable and stored in the header? iam
> > just guessng though, also i dont belive that (x*3)>>2 is correct as its
> > rounding is asymmetric between the positive and negative numbers
>
> If I compare fish05.avi with the Huffman encoded variants at
>
> http://tranquillity.campus.luth.se/~banan/indeo2/
>
> then I believe that the artifacts are already present in the original or
> at least very hard to differentiate from what the original looks like.
>
> I vote for applying Kostya's patch.  It's much better than what we have
> currently and a perfect solution is unlikely to appear.

ok, applied

[...]
-- 
Michael