Dropped unnecessary FieldElement.cmov()

str4d · str4d · commit de84365380a9 · 2014-05-05T12:44:29.000Z
diff --git a/src/net/i2p/crypto/eddsa/math/FieldElement.java b/src/net/i2p/crypto/eddsa/math/FieldElement.java
@@ -115,19 +115,6 @@ public FieldElement pow(FieldElement e){
         return pow(e.bi);
     }
 
-    /**
-     * Replace this with other if b == 1.
-     * Replace this with this if b == 0.
-     * @param other
-     * @param b in {0, 1}
-     * @return
-     */
-    public FieldElement cmov(FieldElement other, int b) {
-        BigInteger x = bi.xor(other.bi);
-        x = x.and(BigInteger.valueOf(-b));
-        return new FieldElement(f, bi.xor(x));
-    }
-
     @Override
     public int hashCode() {
         return bi.hashCode();
diff --git a/test/net/i2p/crypto/eddsa/math/FieldElementTest.java b/test/net/i2p/crypto/eddsa/math/FieldElementTest.java
@@ -145,19 +145,6 @@ public void testModPow() {
         fail("Not yet implemented");
     }
 
-    /**
-     * Test method for {@link FieldElement#cmov(FieldElement, int)}.
-     */
-    @Test
-    public void testCmov() {
-        assertThat(ZERO.cmov(ONE, 0), is(equalTo(ZERO)));
-        assertThat(ZERO.cmov(ONE, 1), is(equalTo(ONE)));
-        FieldElement five = new FieldElement(ed25519Field, Constants.FIVE);
-        FieldElement notfive = new FieldElement(ed25519Field, BigInteger.valueOf(321));
-        assertThat(five.cmov(notfive, 0), is(five));
-        assertThat(five.cmov(notfive, 1), is(notfive));
-    }
-
     /**
      * Test method for {@link FieldElement#equals(java.lang.Object)}.
      */
