This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH] Fix parameter type in C++ version of iseqsig (bug 23171)

From: "Gabriel F. T. Gomes" <gabriel at inconstante dot eti dot br>
To: Joseph Myers <joseph at codesourcery dot com>
Cc: <libc-alpha at sourceware dot org>
Date: Thu, 24 May 2018 15:48:53 -0300
Subject: Re: [PATCH] Fix parameter type in C++ version of iseqsig (bug 23171)
References: <20180514142858.26953-1-gabriel@inconstante.eti.br> <alpine.DEB.2.20.1805141514490.28383@digraph.polyomino.org.uk>

On Mon, 14 May 2018, Joseph Myers wrote:

>On Mon, 14 May 2018, Gabriel F. T. Gomes wrote:
>
>> The commit
>> 
>>   commit c85e54ac6cef0faed7b7ffc722f52523dec59bf5
>>   Author: Gabriel F. T. Gomes <gabriel@inconstante.eti.br>
>>   Date:   Fri Nov 3 10:44:36 2017 -0200
>> 
>>       Provide a C++ version of iseqsig (bug 22377)
>> 
>> mistakenly used double parameters in the long double version of iseqsig,
>> thus causing spurious conversions to double, as reported on bug 23171.
>> 
>> Tested for powerpc64le and x86_64.
>> 
>> 	[BZ #23171]
>> 	* math/math.h [C++] (iseqsig): Fix parameter type for the long
>> 	double version.  
>
>OK.

Thanks.  Now pushed as fb0e10b8eb1e.  I'll wait some days, then I can
backport this fix to 2.26 and 2.27.

References:
- [PATCH] Fix parameter type in C++ version of iseqsig (bug 23171)
  - From: Gabriel F. T. Gomes
- Re: [PATCH] Fix parameter type in C++ version of iseqsig (bug 23171)
  - From: Joseph Myers

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]