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

Reimar Döffinger Reimar.Doeffinger
Wed Nov 4 18:30:28 CET 2009


On Wed, Nov 04, 2009 at 04:50:21PM +0100, Michael Niedermayer wrote:
> On Wed, Nov 04, 2009 at 10:16:08AM +0100, Reimar D?ffinger wrote:
> > 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

Applied.



More information about the ffmpeg-devel mailing list