[FFmpeg-devel] [PATCH] get rid of ff_cos_tabs and ff_sin_tabs offsets

Michael Niedermayer michaelni
Wed Nov 4 16:50:21 CET 2009


On Wed, Nov 04, 2009 at 10:16:08AM +0100, Reimar D?ffinger wrote:
> Hello,
> currently ff_cos_tabs and ff_sin_tabs are indexed as [nr_bits - 4].
> Attached patch instead adds NULL padding to get rid of that offset.
> Obviously it wastes up to 64 bytes, but the slightly less confusing
> code seems worth it to me, particularly since I intend to add
> a function to initialize a specific element of those tables, where
> it would be non-obvious if the argument is the index or the "bits".

>  dsputil.h |    2 +-
>  fft.c     |    3 ++-
>  rdft.c    |    5 +++--
>  3 files changed, 6 insertions(+), 4 deletions(-)
> 488aa023bf951ace8eccc2c96c0d16a7442675a6  taboffset.diff

ok

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have never wished to cater to the crowd; for what I know they do not
approve, and what they approve I do not know. -- Epicurus
-------------- 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/20091104/7a0ea712/attachment.pgp>



More information about the ffmpeg-devel mailing list