You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by er...@apache.org on 2019/09/29 14:07:22 UTC

[commons-numbers] 02/06: Useless use of "return" and braces.

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

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

commit 3b0c9883a737f3057494c785d1271956440b0b62
Author: Gilles Sadowski <gi...@harfang.homelinux.org>
AuthorDate: Sun Sep 29 15:47:43 2019 +0200

    Useless use of "return" and braces.
    
    Reported by "SonarQube".
---
 .../commons/numbers/quaternion/Quaternion.java      | 21 ++++++++++-----------
 1 file changed, 10 insertions(+), 11 deletions(-)

diff --git a/commons-numbers-quaternion/src/main/java/org/apache/commons/numbers/quaternion/Quaternion.java b/commons-numbers-quaternion/src/main/java/org/apache/commons/numbers/quaternion/Quaternion.java
index 859cf78..08683d2 100644
--- a/commons-numbers-quaternion/src/main/java/org/apache/commons/numbers/quaternion/Quaternion.java
+++ b/commons-numbers-quaternion/src/main/java/org/apache/commons/numbers/quaternion/Quaternion.java
@@ -88,25 +88,24 @@ public final class Quaternion implements Serializable {
         /** Default implementations. */
         private static final class Default {
             /** {@link Quaternion#normSq()}. */
-            static final ToDoubleFunction<Quaternion> NORMSQ = q -> {
-                return q.w * q.w + q.x * q.x + q.y * q.y + q.z * q.z;
-            };
+            static final ToDoubleFunction<Quaternion> NORMSQ = q ->
+                q.w * q.w + q.x * q.x + q.y * q.y + q.z * q.z;
+
             /** {@link Quaternion#norm()}. */
-            private static final ToDoubleFunction<Quaternion> NORM = q -> {
-                return Math.sqrt(NORMSQ.applyAsDouble(q));
-            };
+            private static final ToDoubleFunction<Quaternion> NORM = q ->
+                Math.sqrt(NORMSQ.applyAsDouble(q));
+
             /** {@link Quaternion#isUnit()}. */
-            private static final BiPredicate<Quaternion, Double> IS_UNIT = (q, eps) -> {
-                return Precision.equals(NORM.applyAsDouble(q), 1d, eps);
-            };
-        };
+            private static final BiPredicate<Quaternion, Double> IS_UNIT = (q, eps) ->
+                Precision.equals(NORM.applyAsDouble(q), 1d, eps);
+        }
         /** Implementations for normalized quaternions. */
         private static final class Normalized {
             /** {@link Quaternion#norm()} returns 1. */
             static final ToDoubleFunction<Quaternion> NORM = q -> 1;
             /** {@link Quaternion#isUnit()} returns 1. */
             static final BiPredicate<Quaternion, Double> IS_UNIT = (q, eps) -> true;
-        };
+        }
 
         /** {@link Quaternion#normSq()}. */
         private final ToDoubleFunction<Quaternion> normSq;