[FFmpeg-devel] [PATCH] hardcoded ff_cos tables

Reimar Döffinger Reimar.Doeffinger
Wed Oct 14 20:01:06 CEST 2009


On Wed, Oct 14, 2009 at 07:15:58PM +0200, Reimar D?ffinger wrote:
> > As for the need, everything with external linkage should be
> > declared in some header IMHO.  Furthermore, someone might want to use
> > those tables from C code too, not just asm.
> 
> Which is a completely unrelated, already existing mess.
> And no, I can't see a reason to use those tables from C code, that's
> what ff_cos_tab is there for (and it _might_ even make sense to use it
> from asm code).

This patch should demonstrate it is an independent issue, it needs no
extra header file and eliminates a good bit of code duplication.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: costable.diff
Type: text/x-diff
Size: 5528 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20091014/16980e97/attachment.diff>



More information about the ffmpeg-devel mailing list