This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PING][PATCH][BZ #6803] Set errno for scalbln
- From: Siddhesh Poyarekar <siddhesh dot poyarekar at gmail dot com>
- To: "Joseph S. Myers" <joseph at codesourcery dot com>
- Cc: Siddhesh Poyarekar <siddhesh at redhat dot com>, Stefan Liebler <stli at linux dot vnet dot ibm dot com>, GNU C Library <libc-alpha at sourceware dot org>
- Date: Fri, 20 Jun 2014 17:12:35 +0530
- Subject: Re: [PING][PATCH][BZ #6803] Set errno for scalbln
- Authentication-results: sourceware.org; auth=none
- References: <Pine dot LNX dot 4 dot 64 dot 1405071534410 dot 29322 at digraph dot polyomino dot org dot uk> <lkfccv$gd5$1 at ger dot gmane dot org> <mvmvbtgy9oo dot fsf at hawking dot suse dot de> <lkfg3q$6sc$1 at ger dot gmane dot org> <mvmr444y6u3 dot fsf at hawking dot suse dot de> <lkflha$qha$1 at ger dot gmane dot org> <mvmmwesy3e9 dot fsf at hawking dot suse dot de> <lkg4fo$itv$1 at ger dot gmane dot org> <ll20fd$8uf$1 at ger dot gmane dot org> <Pine dot LNX dot 4 dot 64 dot 1406192158030 dot 2033 at digraph dot polyomino dot org dot uk> <20140620023410 dot GS7238 at spoyarek dot pnq dot redhat dot com> <Pine dot LNX dot 4 dot 64 dot 1406201131080 dot 10144 at digraph dot polyomino dot org dot uk>
Sorry about that, I've reverted the NEWS change and also fixed up the
ChangeLog goof-up with the localedata fix.
Siddhesh
On 20 June 2014 17:02, Joseph S. Myers <joseph@codesourcery.com> wrote:
> On Fri, 20 Jun 2014, Siddhesh Poyarekar wrote:
>
>> On Thu, Jun 19, 2014 at 09:58:33PM +0000, Joseph S. Myers wrote:
>> > On Thu, 15 May 2014, Stefan Liebler wrote:
>> >
>> > > PING!
>> > >
>> > > This patch introduces a new wrapper for scalbln functions
>> > > in order to set errno to ERANGE on overflow and underflow.
>> > > The existing aliases that uses __scalbln functions are replaced by aliases to
>> > > the new wrapper.
>> > > The corresponding tests in libm-test.inc are updated to check for errno via
>> > > ERRNO_PLUS_OFLOW etc.
>> >
>> > This patch is OK.
>>
>> Pushed.
>
> Bug 6803 should not have been added to NEWS; this patch only fixes the
> scalbln case, not the scalbn one. (I previously fixed the scalb case.)
>
> --
> Joseph S. Myers
> joseph@codesourcery.com
--
http://siddhesh.in