[CELT-dev] Another thing it might be nice to fix in CELT
jridges at masque.com
Tue Mar 1 08:59:20 PST 2011
In the compiler I'm using, Visual Studio 2005 express, the intrinsic
_BitScanReverse (used in ecintrin.h) isn't defined for a debug build, so
I have to use ec_ilog for the definition of EC_ILOG. Now maybe that's
just because I'm using an old compiler and it's not something you need
to worry about. However, it did point out a problem that if you *do*
have to use the default "# define EC_ILOG(_x) (ec_ilog(_x))" definition
of EC_ILOG, you get warnings in the entcode.h header file that "ec_ilog"
is undefined. That's because the prototype for "ec_ilog" is *after* a
reference to EC_ILOG in that header file. I fixed it by just moving the
prototype for "ec_ilog" higher up in entcode.h.
More information about the celt-dev