You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Alex Herbert (Jira)" <ji...@apache.org> on 2022/10/14 11:34:00 UTC
[jira] [Reopened] (NUMBERS-48) Unreachable statements in Complex.abs()
[ https://issues.apache.org/jira/browse/NUMBERS-48?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Alex Herbert reopened NUMBERS-48:
---------------------------------
> Unreachable statements in Complex.abs()
> ---------------------------------------
>
> Key: NUMBERS-48
> URL: https://issues.apache.org/jira/browse/NUMBERS-48
> Project: Commons Numbers
> Issue Type: Bug
> Reporter: David Nickerson
> Priority: Minor
> Labels: easyfix, newbie, patch
> Fix For: 1.0-beta1
>
> Attachments: complex_abs.patch
>
>
> This return statement in Complex.abs() is unreachable:
> {code:java}
> if (FastMath.abs(real) < FastMath.abs(imaginary)) {
> if (imaginary == 0.0) {
> return FastMath.abs(real);
> }
> {code}
> If imaginary == 0, then there's no way that the preceding condition would be true. There are two similar inner 'if' statements that were accidentally switched. Returned values are still correct, but performance suffers.
> The attached patch switches these back. Note that we're still protected from dividing by zero.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)