[FFmpeg-devel] [PATCH 2/2] Remove ff_inverse from libavcodec.

[Diego 'Flameeyes' Pettenò]

Uoti Urpala <uoti.urpala at pp1.inet.fi> writes:

> Using a symbol from another library is slower (a symbol from the same
> library is at a constant offset, a symbol from another library requires
> lookup through offset table). Since the whole point of FASTDIV is speed
> this could matter.

Yeah I expected something like this. An altenative approach would be to
check for CONFIG_SMALL: if it's not defined, ff_inverse could be defined
static inside the ff_sqrt function itself, otherwise the external
declaration would be used. The problem is that it would make the header
file dependent on a build configuration option.

Comments?

-- 
Diego "Flameeyes" Petten?
http://blog.flameeyes.eu/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 196 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081002/3e73397b/attachment.pgp>