[FFmpeg-devel] [PATCH 2/3] Indeo 5 decoder: common DSP functions

Kostya kostya.shishkov
Sat Jan 9 15:40:30 CET 2010


On Fri, Jan 08, 2010 at 11:41:23PM +0100, Michael Niedermayer wrote:
> On Sun, Jan 03, 2010 at 12:56:36PM +0200, Kostya wrote:
> [...]
> > void ff_ivi_recompose53(const IVIPlaneDesc *plane, uint8_t *dst,
[function body skipped]
> 
> is this mess faster than some more readable variant?

Here's more readable variant by me, checked to be bitexact but it's
significantly slower (> 10%), I'd rather leave old one.
 
> [...]
> > void ff_ivi_mc_8x8_no_delta(int16_t *buf, int16_t *ref_buf, uint32_t pitch, int mc_type)
> 
> the delta and no_delta can be merged i guess with a macro

done
 
> [...]
> -- 
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ivi_dsp.c
Type: text/x-csrc
Size: 13675 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100109/420c11c0/attachment.c>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ivi_dsp.h
Type: text/x-chdr
Size: 6383 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100109/420c11c0/attachment.h>



More information about the ffmpeg-devel mailing list