Re: CVS commit: src/lib/libm/man

2011-09-12 Thread Joerg Sonnenberger
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

2011-09-12 Thread Jukka Ruohonen
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

2011-09-12 Thread Iain Hibbert
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

2011-09-12 Thread Warner Losh
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

2011-09-12 Thread Christos Zoulas
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