[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