This is the mail archive of the newlib@sourceware.org mailing list for the newlib 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] libm/complex: add clog10, clog10f

From: Yaakov Selkowitz <yselkowitz at cygwin dot com>
To: newlib at sourceware dot org
Date: Mon, 22 Feb 2016 14:43:27 -0600
Subject: Re: [PATCH] libm/complex: add clog10, clog10f
Authentication-results: sourceware.org; auth=none
References: <1455814389-12684-1-git-send-email-yselkowi at redhat dot com> <20160222092119 dot GA11862 at calimero dot vinschen dot de> <56CB33DA dot 2090302 at cygwin dot com> <20160222170922 dot GD29199 at calimero dot vinschen dot de>

On 2016-02-22 11:09, Corinna Vinschen wrote:

On Feb 22 10:14, Yaakov Selkowitz wrote:

On 2016-02-22 03:21, Corinna Vinschen wrote:

Did you write these functions by yourself or did you fetch them from
some upstream?  If so, where from?


They are tweaks of our existing clog() and clogf().

My complex math is next to inexistent, but I took a look into the glibc
man page.  The man page says "clog10(z) is equivalent to clog(z)/log(10)".
The above is equivalent, I take it?


https://sourceware.org/bugzilla/show_bug.cgi?id=19673


Erm... sorry but I don't see any mention of atan2 in this BZ.


See newlib/libm/complex/carg.c.

Also, what about Craig's point?


I have already fixed that locally.

--
Yaakov

Follow-Ups:
- Re: [PATCH] libm/complex: add clog10, clog10f
  - From: Corinna Vinschen

References:
- [PATCH] libm/complex: add clog10, clog10f
  - From: Yaakov Selkowitz
- Re: [PATCH] libm/complex: add clog10, clog10f
  - From: Corinna Vinschen
- Re: [PATCH] libm/complex: add clog10, clog10f
  - From: Yaakov Selkowitz
- Re: [PATCH] libm/complex: add clog10, clog10f
  - From: Corinna Vinschen

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