[FFmpeg-devel] [PATCH] Indeo5 decoder

[Michael Niedermayer]

On Wed, Apr 01, 2009 at 01:50:45PM +0200, Maxim wrote:
> Michael Niedermayer schrieb:
> > On Wed, Apr 01, 2009 at 02:43:04AM +0200, Maxim wrote:
> >   
> >> Diego Biurrun schrieb:
> >>     
> >>> Have you checked this works standalone, i.e. there are no other
> >>> dependencies?
> >>>
> >>> Try disabling all decoders and encoders and just enabling indeo5 please.
> >>>   
> >>>       
> >> Yes, it does...
> >>
> >> Attached is an updated patch that fixes all things you pointed to.
> >> The following things were changed as well:
> >> - the inverse transform contains algebraic simplifications showed by
> >> Michael in his earlier post
> >> - a step was added to the inverse transform that tests for col/rows
> >> containing zero coeffs and sets the corresponding output to zero instead
> >> of performing the inverse transform on zeroes
> >> - simplifications of the motion compensation functions
> >>
> >> Please any further reviews (Michael?)
> >>     
> > [snip]
> >   
> 
> > Non static with no ff_/av_ prefix
> > indeo5patch.txt:856:+AVCodec indeo5_decoder = {
> >   
> I'm sorry, where should I add ff_/av_ prefix here?

ff_indeo5_decoder

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Dictatorship naturally arises out of democracy, and the most aggravated
form of tyranny and slavery out of the most extreme liberty. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090401/6b5555d7/attachment.pgp>