You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by ah...@apache.org on 2019/12/09 12:06:17 UTC

[commons-numbers] branch master updated: sqrt(): Drop copySign(0, imaginary) when imaginary is already zero.

This is an automated email from the ASF dual-hosted git repository.

aherbert pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-numbers.git


The following commit(s) were added to refs/heads/master by this push:
     new 0887c25  sqrt(): Drop copySign(0, imaginary) when imaginary is already zero.
0887c25 is described below

commit 0887c257a298d261204d35a311b1a5900812657c
Author: aherbert <ah...@apache.org>
AuthorDate: Mon Dec 9 12:06:13 2019 +0000

    sqrt(): Drop copySign(0, imaginary) when imaginary is already zero.
---
 .../src/main/java/org/apache/commons/numbers/complex/Complex.java       | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/commons-numbers-complex/src/main/java/org/apache/commons/numbers/complex/Complex.java b/commons-numbers-complex/src/main/java/org/apache/commons/numbers/complex/Complex.java
index 4459dc4..d4ddf26 100644
--- a/commons-numbers-complex/src/main/java/org/apache/commons/numbers/complex/Complex.java
+++ b/commons-numbers-complex/src/main/java/org/apache/commons/numbers/complex/Complex.java
@@ -1709,7 +1709,7 @@ public final class Complex implements Serializable  {
                     if (real < 0) {
                         return new Complex(0, Math.copySign(sqrtAbs, imaginary));
                     }
-                    return new Complex(sqrtAbs, Math.copySign(0, imaginary));
+                    return new Complex(sqrtAbs, imaginary);
                 }
                 final double abs = getAbsolute(real, imaginary);
                 final double av = (Math.abs(real) + abs) / 2;