[FFmpeg-cvslog] r23647 - in trunk: Changelog libavcodec/Makefile libavcodec/aacdec.c libavcodec/aacsbr.c libavcodec/avcodec.h libavcodec/mpeg4audio.c libavcodec/ps.c libavcodec/ps.h libavcodec/ps_tablegen.c libavc...

Michael Niedermayer michaelni
Sun Jun 20 21:52:56 CEST 2010


On Sun, Jun 20, 2010 at 03:10:48PM +0200, Reimar D?ffinger wrote:
> On Sun, Jun 20, 2010 at 02:53:36PM +0200, Reimar D?ffinger wrote:
> > On Sun, Jun 20, 2010 at 02:39:28PM +0200, Michael Niedermayer wrote:
> > > On Sun, Jun 20, 2010 at 08:27:17AM +0200, Reimar D?ffinger wrote:
> > > > On Sun, Jun 20, 2010 at 04:06:06AM +0200, Michael Niedermayer wrote:
> > > > > > +#if CONFIG_HARDCODED_TABLES
> > > > > > +#define ps_tableinit()
> > > > > > +#include "libavcodec/ps_tables.h"
> > > > > > +#else
> > > > > > +#include "../libavutil/common.h"
> > > > > > +#ifndef M_SQRT1_2
> > > > > > +#define M_SQRT1_2      0.70710678118654752440  /* 1/sqrt(2) */
> > > > > > +#endif
> > > > > > +#ifndef M_PI
> > > > > > +#define M_PI           3.14159265358979323846  /* pi */
> > > > > > +#endif
> > > > > > +#ifndef M_SQRT2
> > > > > > +#define M_SQRT2        1.41421356237309504880  /* sqrt(2) */
> > > > > 
> > > > > these constants should be already in some libavutil header
> > > > 
> > > > Yes, but that header can't be used for host compilation.
> > > 
> > > why not?
> > 
> > Sorry! That was based on outdated information.
> > Yes, mathematics.h should be safe to use, so it should be used...
> 
> Separate header or not, I can suggest patch below...

>  doc/tablegen.txt                |    1 +
>  libavcodec/mdct_tablegen.h      |    2 +-
>  libavcodec/mpegaudio_tablegen.h |    2 --
>  libavcodec/ps_tablegen.h        |   11 +----------
>  4 files changed, 3 insertions(+), 13 deletions(-)
> 123554fb4402002ea828ca587ba4039ee8b70b07  tablegen.diff

non ps ok

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

The misfortune of the wise is better than the prosperity of the fool.
-- 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-cvslog/attachments/20100620/bc898ee2/attachment-0001.pgp>



More information about the ffmpeg-cvslog mailing list