8333046: Clean codes in sun.security.util.math

johnshajiang · May 28, 2024 · bac6397 · bac6397
1 parent 87a06b6
commit bac6397
Show file tree

Hide file tree

Showing 6 changed files with 6 additions and 9 deletions.
diff --git a/src/java.base/share/classes/sun/security/util/math/IntegerMontgomeryFieldModuloP.java b/src/java.base/share/classes/sun/security/util/math/IntegerMontgomeryFieldModuloP.java
@@ -25,8 +25,6 @@
 
 package sun.security.util.math;
 
-import java.math.BigInteger;
-
 /**
  * An interface for the field of integers modulo a prime number. An
  * implementation of this interface can be used to get properties of the

diff --git a/src/java.base/share/classes/sun/security/util/math/intpoly/IntegerPolynomial.java b/src/java.base/share/classes/sun/security/util/math/intpoly/IntegerPolynomial.java
@@ -824,7 +824,7 @@ public MutableElement setSquare() {
                 numAdds = 0;
             }
 
-            numAdds = IntegerPolynomial.this.square(limbs, limbs);;
+            numAdds = IntegerPolynomial.this.square(limbs, limbs);
             return this;
         }
 

diff --git a/...va.base/share/classes/sun/security/util/math/intpoly/MontgomeryIntegerPolynomialP256.java b/...va.base/share/classes/sun/security/util/math/intpoly/MontgomeryIntegerPolynomialP256.java
@@ -29,7 +29,6 @@
 import sun.security.util.math.IntegerMontgomeryFieldModuloP;
 import sun.security.util.math.SmallValue;
 import sun.security.util.math.IntegerFieldModuloP;
-import java.lang.Math;
 import java.math.BigInteger;
 import jdk.internal.vm.annotation.IntrinsicCandidate;
 
@@ -110,7 +109,7 @@ public ImmutableElement getElement(BigInteger v) {
 
     @Override
     public SmallValue getSmallValue(int value) {
-        // Explicitely here as reminder that SmallValue stays in residue domain
+        // Explicitly here as reminder that SmallValue stays in residue domain
         // See multByInt below for how this is used
         return super.getSmallValue(value);
     }
@@ -557,4 +556,4 @@ protected void reduceIn(long[] limbs, long v, int i) {
         limbs[i - 5] += (v << 4) & LIMB_MASK;
         limbs[i - 4] += v >> 48;
     }
-}
+}
diff --git a/test/jdk/com/sun/security/ec/ECOperationsKATTest.java b/test/jdk/com/sun/security/ec/ECOperationsKATTest.java
@@ -250,4 +250,4 @@ int main(){
   BN_CTX_free(ctx);
   return 0;
 }
- */
+ */
diff --git a/test/jdk/com/sun/security/util/math/intpoly/IntegerPolynomialTest.java b/test/jdk/com/sun/security/util/math/intpoly/IntegerPolynomialTest.java
@@ -92,4 +92,4 @@ public static void main(String[] args) throws Exception {
     }
 }
 
-//make test TEST="test/jdk/com/sun/security/util/math/intpoly/IntegerPolynomialTest.java"
+//make test TEST="test/jdk/com/sun/security/util/math/intpoly/IntegerPolynomialTest.java"
diff --git a/test/jdk/com/sun/security/util/math/intpoly/MontgomeryPolynomialFuzzTest.java b/test/jdk/com/sun/security/util/math/intpoly/MontgomeryPolynomialFuzzTest.java
@@ -97,4 +97,4 @@ public static void run() throws Exception {
     }
 }
 
-//make test TEST="test/jdk/com/sun/security/util/math/intpoly/MontgomeryPolynomialFuzzTest.java"
+//make test TEST="test/jdk/com/sun/security/util/math/intpoly/MontgomeryPolynomialFuzzTest.java"
-Original file line number
+Diff line change
@@ Expand Up / @@ -250,4 +250,4 @@ int main(){ @@
       BN_CTX_free(ctx);
       return 0;
     }
-     */
+     */