[uClibc]math library ... e_asin.c gcc warning ...
Erik Andersen
andersen at codepoet.org
Thu May 9 03:51:48 UTC 2002
On Wed May 08, 2002 at 12:17:01PM -0700, m4 at brecis.COM wrote:
> File libm/e_asin.c
>
> e_asin.c: In function `__ieee754_asin':
> e_asin.c:80: warning: `t' might be used uninitialized in this function
>
> ...
> } else
> t = x*x;
> p = t*(pS0+t*(pS1+t*(pS2+t*(pS3+t*(pS4+t*pS5)))));
> q = one+t*(qS1+t*(qS2+t*(qS3+t*qS4)));
> w = p/q;
> return x+x*w;
> }
> ...
>
> I do not know the algorithm, but the above indentation implies
> a missing brace pair. Everything else sets t correctly.
This is identical to the version from fdlibm:
http://www.netlib.org/fdlibm/e_asin.c
And after carefully reading this bit of code over several times,
I feel certain that adding the missing braces is the Right
Thing(tm)...
-Erik
--
Erik B. Andersen http://codepoet-consulting.com/
--This message was written using 73% post-consumer electrons--
More information about the uClibc
mailing list