[xiph-commits] r13629 - in trunk/vorbis: . lib

xiphmont at xiph.org xiphmont at xiph.org
Tue Aug 28 10:51:13 PDT 2007


On 8/25/07, erikd at svn.xiph.org <erikd at svn.xiph.org> wrote:
> Author: erikd
> Date: 2007-08-25 17:24:57 -0700 (Sat, 25 Aug 2007)
> New Revision: 13629
>
> Modified:
>    trunk/vorbis/configure.in
>    trunk/vorbis/lib/scales.h
> Log:
> Detect availability of C99 inline keyword in configure and use it in lib/scales.h. Also add workaround for MSVS which uses __inline.

erikd, does this show any demonstrable benefit?  -O3 forces inlining
of the functions you modified anyway, and once we're not on MSVC or
GCC, inline is not portable.  I worry this is just a redundant
restriction that adds portability worries.

Monty


More information about the commits mailing list