diff --git a/acl/src/config.rs b/acl/src/config.rs
index 7b48d6b..2f0f8a9 100644
--- a/acl/src/config.rs
+++ b/acl/src/config.rs
@@ -54,7 +54,7 @@ impl<A: ACLConfig> KeyPair<A> {
     pub fn affine_from_bytes_tai(bytes: &[u8]) -> sw::Affine<A> {
         use sha3::{Digest, Sha3_256};
 
-        for i in 0..=u8::max_value() {
+        for i in 0..=u8::MAX {
             let mut sha = Sha3_256::new();
             Digest::update(&mut sha, bytes);
             Digest::update(&mut sha, [i]);
diff --git a/boomerang/src/server.rs b/boomerang/src/server.rs
index 1f5e615..6911b60 100644
--- a/boomerang/src/server.rs
+++ b/boomerang/src/server.rs
@@ -393,8 +393,8 @@ pub struct SpendVerifyM3<B: BoomerangConfig> {
 impl<B: BoomerangConfig> Clone for SpendVerifyM3<B> {
     fn clone(&self) -> Self {
         Self {
-            comm: self.comm.clone(),
-            sig_commit: self.sig_commit.clone(),
+            comm: self.comm,
+            sig_commit: self.sig_commit,
             id_1: self.id_1,
             verifying_key: self.verifying_key,
             tag_key: self.tag_key,
diff --git a/pedersen/src/pedersen_config.rs b/pedersen/src/pedersen_config.rs
index 5380726..887d6da 100644
--- a/pedersen/src/pedersen_config.rs
+++ b/pedersen/src/pedersen_config.rs
@@ -455,7 +455,7 @@ impl<P: PedersenConfig> PedersenComm<P> {
     pub fn affine_from_bytes_tai(bytes: &[u8]) -> sw::Affine<P> {
         use sha3::{Digest, Sha3_256};
         // Try a deterministic sequence of hashes to find a valid point.
-        for i in 0..=u8::max_value() {
+        for i in 0..=u8::MAX {
             let mut sha = Sha3_256::new();
             Digest::update(&mut sha, bytes);
             Digest::update(&mut sha, [i]);