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