[Speex-dev] Speex with PS3 SPE support

[Jean-Marc Valin]

Hi Jim,

Jim Crichton wrote:
> Please correct me if I am wrong, Jean-Marc, but I do not think that
> any patches are getting applied to 1.0.5 anyway.  Also, if you expect
> a patch to be applied, you will need to provide the changes as a
> patch, not as a modified copy of the source tree.

That's right. I won't be applying anything 1.0.5 unless it's a security
issue or something like that.

> The 1.2 branch includes a mechanism for private memory allocation
> from a static buffer.  You provide a usermisc.h file that replaces
> the normal alloc routines.  You should be able to fix most if not all
> of your alignment issues there, rather than putting conditionals in
> the main code.   I do not know if it will allow you to avoid the
> __attribute__ directives.  Look at the TI subdirectory in the main
> tree for an example of how this is done. There is a bit of a hack for
> providing a private stack without changing the API.  I do not think
> that there is any precedent for platform specific API changes, such
> as what you have proposed.

That reminds me that I've just moved lots of stuff around with the
allocation functions. Can you check that the TI stuff still works with
that? In the end, it'll probably make things easier for you now. For
example, if you link statically, wideband should be automatically left
out if unused. Also, all the libc stuff is now in os_support.h, which is
less messy than misc.h/misc.c were.

Cheers,

	Jean-Marc