public inbox for libc-help@sourceware.org
 help / color / mirror / Atom feed
* Elementary Functions
@ 2020-01-13 15:09 N.G. Timmons
  2020-01-13 16:49 ` Carlos O'Donell
  0 siblings, 1 reply; 2+ messages in thread
From: N.G. Timmons @ 2020-01-13 15:09 UTC (permalink / raw)
  To: libc-help

Hi,

I was looking through the source for sin/cos/exp etc. and noticed that a 
lot of the comments don't actually say what/why things are being done.
There also exists a union type called "mynumber" which is used to access 
the high and low bits of a 64-bit floating point number.

I would like to comment the elementary functions with:
     - What algorithm is being used
     - Why is it being used
     - Any caveats on unusual operations so future changes can be made 
safely
     - Actual testing information (the current statistics given at the 
top of some functions doesn't say how they were tested). This is to 
allow for verification and comparison if future changes are made.

I would like to rename 'mynumber' and a few other variables to 
meaningful names so that the source code can be followed with greater 
ease.

Would the community be open to me making these changes?

Thanks,
Nick Timmons

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: Elementary Functions
  2020-01-13 15:09 Elementary Functions N.G. Timmons
@ 2020-01-13 16:49 ` Carlos O'Donell
  0 siblings, 0 replies; 2+ messages in thread
From: Carlos O'Donell @ 2020-01-13 16:49 UTC (permalink / raw)
  To: N.G. Timmons; +Cc: libc-help

On Mon, Jan 13, 2020 at 10:09 AM N.G. Timmons <ngt26@cam.ac.uk> wrote:
>
> Hi,
>
> I was looking through the source for sin/cos/exp etc. and noticed that a
> lot of the comments don't actually say what/why things are being done.
> There also exists a union type called "mynumber" which is used to access
> the high and low bits of a 64-bit floating point number.
>
> I would like to comment the elementary functions with:
>      - What algorithm is being used
>      - Why is it being used
>      - Any caveats on unusual operations so future changes can be made
> safely
>      - Actual testing information (the current statistics given at the
> top of some functions doesn't say how they were tested). This is to
> allow for verification and comparison if future changes are made.
>
> I would like to rename 'mynumber' and a few other variables to
> meaningful names so that the source code can be followed with greater
> ease.
>
> Would the community be open to me making these changes?

I think such changes would be great.

The various elementary math functions are part of glibc and we don't
import that code from anywhere anymore
(https://sourceware.org/glibc/wiki/SharedSourceFiles).

Joseph Myers does a lot of review of these functions and given your
email address I expect you might know him? :-)

Please post to libc-alpha@sourceware.org so core developers can see your post.

Usually libc-help is for help of one kind or another, and your
question is more directly related to developers.

Thanks!

Cheers,
Carlos.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2020-01-13 16:49 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-01-13 15:09 Elementary Functions N.G. Timmons
2020-01-13 16:49 ` Carlos O'Donell

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).