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 2020/06/30 13:38:25 UTC

[commons-numbers] 03/03: Remove extra if statement.

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

commit 6b0e82ae718dc17ab163b3949ba9f2d821a42d9d
Author: aherbert <ah...@apache.org>
AuthorDate: Tue Jun 30 14:15:39 2020 +0100

    Remove extra if statement.
---
 .../src/main/java/org/apache/commons/numbers/primes/Primes.java      | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/commons-numbers-primes/src/main/java/org/apache/commons/numbers/primes/Primes.java b/commons-numbers-primes/src/main/java/org/apache/commons/numbers/primes/Primes.java
index b135f4b..860678d 100644
--- a/commons-numbers-primes/src/main/java/org/apache/commons/numbers/primes/Primes.java
+++ b/commons-numbers-primes/src/main/java/org/apache/commons/numbers/primes/Primes.java
@@ -70,13 +70,10 @@ public final class Primes {
         if (n < 0) {
             throw new IllegalArgumentException(MessageFormat.format(NUMBER_TOO_SMALL, n, 0));
         }
-        if (n == 2) {
+        if (n <= 2) {
             return 2;
         }
         n |= 1; // make sure n is odd
-        if (n == 1) {
-            return 2;
-        }
 
         if (isPrime(n)) {
             return n;