[CELT-dev] Another thing it might be nice to fix in CELT

[John Ridges]

Hi Jean-Marc,

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.

Cheers,
John Ridges