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/lib/libm/man

2011-09-17 Thread Thomas Klausner
On Sat, Sep 17, 2011 at 10:52:52AM +, Jukka Ruohonen wrote:
> Module Name:  src
> Committed By: jruoho
> Date: Sat Sep 17 10:52:52 UTC 2011
> 
> Modified Files:
>   src/lib/libm/man: exp.3
> 
> Log Message:
> Remove rest of the NOTES and ERRORS and references to pow(3) family.

Why did you remove so much of the explanations in the man page?
 Thomas


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

2011-09-17 Thread Jukka Ruohonen
On Sat, Sep 17, 2011 at 03:32:22PM +0200, Thomas Klausner wrote:
> > Remove rest of the NOTES and ERRORS and references to pow(3) family.
> 
> Why did you remove so much of the explanations in the man page?

Because much of it was outdated or incomplete, and the rest was what others
have criticized of being "man page spam" (i.e. implementation details).

- Jukka.


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

2017-09-28 Thread Joerg Sonnenberger
On Thu, Sep 28, 2017 at 11:09:05AM +, Maya Rashish wrote:
> Module Name:  src
> Committed By: maya
> Date: Thu Sep 28 11:09:05 UTC 2017
> 
> Modified Files:
>   src/lib/libm/man: copysign.3
> 
> Log Message:
> Remove CAVEATS section
> 
> The man page for an individual function is not the right place to discuss
> the issues with negative zero

I disagree in this specific case. copysign(3) is very much concerned
about the behavior of negative zero, since it fundamentally affects the
behavior. Few other functions care since they don't have a pole at 0.

Joerg


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

2017-09-28 Thread maya
On Thu, Sep 28, 2017 at 03:52:00PM +0200, Joerg Sonnenberger wrote:
> On Thu, Sep 28, 2017 at 11:09:05AM +, Maya Rashish wrote:
> > Module Name:src
> > Committed By:   maya
> > Date:   Thu Sep 28 11:09:05 UTC 2017
> > 
> > Modified Files:
> > src/lib/libm/man: copysign.3
> > 
> > Log Message:
> > Remove CAVEATS section
> > 
> > The man page for an individual function is not the right place to discuss
> > the issues with negative zero
> 
> I disagree in this specific case. copysign(3) is very much concerned
> about the behavior of negative zero, since it fundamentally affects the
> behavior. Few other functions care since they don't have a pole at 0.

csqrt and clog have a branch cut on it :S
sure, I can revert/rephrase