Re: cluts review

[Rich Felker <dalias@aerifal.cx>]

On Wed, Jul 13, 2011 at 06:55:01PM +0200, Luka Marčetić wrote:
> that gets executed with each string). The huge long long integer
> printout is I guess what makes it look as if there are more messages
> than there are. Other than that, it's probably the implementations.

musl definitely has a lot of bugs in this area, and may rewrite the
code (for floating point I will definitely rewrite it because the
current approach is inexact and fundamentally can't be right). But I
think at least the huge long double test is wrong:

strtold(....  offsets endptr by 4940 instead of by 0

As far as I can tell, the string you pass is of the expected form for
the subject sequence. It may overflow to infinity or lose precision,
but it's still valid. strtold is only supposed to store the initial
pointer in endptr if the subject sequence is empty or does not have
the expected form; otherwise it stores a pointer to the byte just past
the end of the subject sequence.

Does this make sense?

Admittedly I need to review the rest of the tests more thoroughly too.

Rich