This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH] Doc fix for 'frexp' in arith.texi
- From: "Carlos O'Donell" <carlos at redhat dot com>
- To: Ben North <ben at redfrontdoor dot org>
- Cc: libc-alpha at sourceware dot org
- Date: Tue, 28 May 2013 17:27:00 -0400
- Subject: Re: [PATCH] Doc fix for 'frexp' in arith.texi
- References: <CAHG0eJUH8FTJmYidHP0nG1bg9brF2QOJ-f4=tNz6NV-uzA=NUw at mail dot gmail dot com>
On 05/26/2013 09:50 AM, Ben North wrote:
> Hi Carlos,
>
>>> The attached patch fixes a slight inaccuracy in the documentation for
>>> 'frexp'.
>>
>> Rewrite using "magnitude" please and as succinctly as possible. [...]
>> Repost with changes and a ChangeLog entry please
>
> Thanks for the feedback. I attach a revised patch and also a changelog
> entry. The patch adds just two words so I hope it meets the
> succinctness criterion :) Also, you wrote one of the words so I agree
> with your assessment with regard to the copyright assignment.
Looks great.
Fixed with this commit.
Very sorry about incorrectly spelling your name in
the commit author information :-(
We need to switch to git-am process workflow for glibc
so I won't make those kinds of mistakes.
commit 56b672e92e10ac2931236dba4e452699fd0d32d1
Author: Ben Norht <ben@redfrontdoor.org>
Date: Tue May 28 17:20:24 2013 -0400
Doc fix for 'frexp' in arith.texi
It is the magnitude of the return value which lies
in [0.5, 1), not the return value itself.
---
2013-05-28 Ben North <ben@redfrontdoor.org>
* manual/arith.texi (frexp): It is the magnitude of the return
value which lies in [0.5, 1), not the return value itself.
Cheers,
Carlos.