Update for clippy

src/key_exchange.rs

@@ -193,7 +193,6 @@ where
 ///
 /// That doc specifies that local is public and remote is private, but seems like
 /// that shouldn't be necessary.
-#[allow(unused)]
 pub fn ecmr_add<C>(local: &PublicKey<C>, remote: &PublicKey<C>) -> Result<PublicKey<C>>
 where
     C: CurveArithmetic,
@@ -207,7 +206,6 @@ where
 // FIXME: subtraction doesn't match up with `jose` for some reason, but appears otherwise correct.
 // My guess is that it isn't actually doing subtraction, but I am not sure what it is doing instead...
 /// Point subtraction, per ECMR in <https://www.mankier.com/1/jose-jwk-exc>
-#[allow(unused)]
 pub fn ecmr_sub<C>(local: &PublicKey<C>, remote: &PublicKey<C>) -> Result<PublicKey<C>>
 where
     C: CurveArithmetic,
@@ -273,6 +271,7 @@ where
     serde_json::from_str(key.to_jwk_string().as_ref()).expect("dependency generated an invalid JWK")
 }
 
+#[allow(unused)]
 fn jwk_from_priv<C>(key: &SecretKey<C>) -> Jwk
 where
     C: CurveArithmetic + JwkParameters,

src/key_exchange_tests.rs

@@ -171,7 +171,7 @@ fn test_roundtrip_full() {
     // x = c + e, this is sent to the server
     let x_pub = ecmr_add(&c_pub, &e_pub).unwrap();
 
-    // y = x & S, server operation, responds with y
+    // y = x * S, server operation, responds with y
     let y_pub = diffie_hellman(&s_priv, &x_pub).unwrap();
 
     // z = s * E
@@ -190,7 +190,7 @@ fn test_roundtrip_jwk() {
 
     // Pretend to be the server
     let server_key_exchange = |x_pub_jwk: &Jwk| -> Result<Jwk> {
-        let x_pub = jwk_to_pub(&x_pub_jwk).unwrap();
+        let x_pub = jwk_to_pub(x_pub_jwk).unwrap();
         let y_pub = diffie_hellman(&s_priv, &x_pub).unwrap();
         Ok(jwk_from_pub(&y_pub))
     };