[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: new modules 'isnan', 'isnanf', 'isnand'
From: |
Ben Pfaff |
Subject: |
Re: new modules 'isnan', 'isnanf', 'isnand' |
Date: |
Tue, 08 Jul 2008 03:56:10 -0700 |
User-agent: |
Gnus/5.11 (Gnus v5.11) Emacs/22.2 (gnu/linux) |
Simon Josefsson <address@hidden> writes:
> Ben Pfaff <address@hidden> writes:
>
>> +Gnulib module: isnan
>>
>> Portability problems fixed by Gnulib:
>> @itemize
>> address@hidden
>> +This macro is missing on some platforms and does not always yield
>> +correct results on others.
>> @end itemize
>
> Hi Ben! A minor comment: If you know which platforms isnan is missing
> on, and which platforms it results in incorrect results, that may be
> useful information to add. Also, knowing which inputs result in
> incorrect output may also be useful. If it is complicated to explain,
> perhaps the self-test or m4 test is a better place for such discussions,
> but I could not find any similar change for that area in your patch (but
> I could have missed it).
Thanks for keeping me honest. I'll definitely improve this
before I push anything. I don't have a good list of platforms on
which isnan is missing, because Bruno's list by platform only
covers functions, but I can do better than what is given here. I
can come up with some platforms that have incorrect results by
merging the docs for isnanf and isnanl.
--
Ben Pfaff
http://benpfaff.org
- new modules 'isnan', 'isnanf', 'isnand', Ben Pfaff, 2008/07/08
- Re: new modules 'isnan', 'isnanf', 'isnand', Simon Josefsson, 2008/07/08
- Re: new modules 'isnan', 'isnanf', 'isnand', Bruno Haible, 2008/07/08
- Re: new modules 'isnan', 'isnanf', 'isnand', Bruno Haible, 2008/07/08
- [patch 2/4] Add isnand module., Ben Pfaff, 2008/07/11
- [patch 1/4] Rename isnand.h to isnand-nolibm.h, similarly for isnanf.h., Ben Pfaff, 2008/07/11
- Re: [patch 1/4] Rename isnand.h to isnand-nolibm.h, similarly for isnanf.h., Bruno Haible, 2008/07/11
- Re: [patch 1/4] Rename isnand.h to isnand-nolibm.h, similarly for isnanf.h., Ben Pfaff, 2008/07/12
- Re: [patch 1/4] Rename isnand.h to isnand-nolibm.h, similarly for isnanf.h., Bruno Haible, 2008/07/12
- [patch 4/4] Add isnan module., Ben Pfaff, 2008/07/11