Re: CVS commit: src/lib/libm/man
On Mon, Sep 12, 2011 at 02:44:27PM +, Jukka Ruohonen wrote: Module Name: src Committed By: jruoho Date: Mon Sep 12 14:44:27 UTC 2011 Modified Files: src/lib/libm/man: ldexp.3 Log Message: Update and improve, and note that the long double variant is not supported. Can we please *not* add such notes? They don't really add value, the lack of a long double prototype at the beginning of the man page is indicator enough. They do add just another thing to keep track of when adding the support though. Joerg
Re: CVS commit: src/lib/libm/man
On Mon, Sep 12, 2011 at 08:00:53PM +0200, Joerg Sonnenberger wrote: On Mon, Sep 12, 2011 at 02:44:27PM +, Jukka Ruohonen wrote: Module Name:src Committed By: jruoho Date: Mon Sep 12 14:44:27 UTC 2011 Modified Files: src/lib/libm/man: ldexp.3 Log Message: Update and improve, and note that the long double variant is not supported. Can we please *not* add such notes? They don't really add value, the lack of a long double prototype at the beginning of the man page is indicator enough. They do add just another thing to keep track of when adding the support though. Fair enough. But I think it is a BUG worth mentioning when an operating system does not support standard C. - Jukka.
Re: CVS commit: src/lib/libm/man
On Mon, 12 Sep 2011, Jukka Ruohonen wrote: On Mon, Sep 12, 2011 at 08:00:53PM +0200, Joerg Sonnenberger wrote: On Mon, Sep 12, 2011 at 02:44:27PM +, Jukka Ruohonen wrote: Module Name: src Committed By: jruoho Date: Mon Sep 12 14:44:27 UTC 2011 Modified Files: src/lib/libm/man: ldexp.3 Log Message: Update and improve, and note that the long double variant is not supported. Can we please *not* add such notes? They don't really add value, the lack of a long double prototype at the beginning of the man page is indicator enough. They do add just another thing to keep track of when adding the support though. Fair enough. But I think it is a BUG worth mentioning when an operating system does not support standard C. you could add a note about such things to src/doc/TODO, or file a PR? iain
Re: CVS commit: src/lib/libm/man
On Sep 12, 2011, at 1:09 PM, Jukka Ruohonen wrote: On Mon, Sep 12, 2011 at 08:00:53PM +0200, Joerg Sonnenberger wrote: On Mon, Sep 12, 2011 at 02:44:27PM +, Jukka Ruohonen wrote: Module Name:src Committed By: jruoho Date: Mon Sep 12 14:44:27 UTC 2011 Modified Files: src/lib/libm/man: ldexp.3 Log Message: Update and improve, and note that the long double variant is not supported. Can we please *not* add such notes? They don't really add value, the lack of a long double prototype at the beginning of the man page is indicator enough. They do add just another thing to keep track of when adding the support though. Fair enough. But I think it is a BUG worth mentioning when an operating system does not support standard C. Then create a bunch of MLINKS to a generic 'This function isn't supported' page. Since the implementors would need to create a real man page or real link, this seems to be to be less likely to cause confusion than notes everywhere that grow stale. Warner
Re: CVS commit: src/tests/lib/libm
In article 20110912162837.5501917...@cvs.netbsd.org, Jukka Ruohonen source-changes-d@NetBSD.org wrote: -=-=-=-=-=- Module Name: src Committed By: jruoho Date: Mon Sep 12 16:28:37 UTC 2011 Modified Files: src/tests/lib/libm: t_ldexp.c t_scalbn.c t_tanh.c Log Message: Happiness of VAX implies ugliness of the code. To generate a diff of this commit: cvs rdiff -u -r1.2 -r1.3 src/tests/lib/libm/t_ldexp.c \ src/tests/lib/libm/t_scalbn.c cvs rdiff -u -r1.4 -r1.5 src/tests/lib/libm/t_tanh.c Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files. We should probably grow a __HAVE_IEEE754FP christos