Apply Clippy-suggested fixes.

NLnetLabs · Nov 17, 2023 · 4b1e066 · 4b1e066
1 parent cd9ff7d
commit 4b1e066
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 3 deletions.
diff --git a/src/ca/publication.rs b/src/ca/publication.rs
@@ -335,7 +335,7 @@ impl Query {
             }
         }
 
-        if pdus.get(0) == Some(&QueryPdu::List) {
+        if pdus.first() == Some(&QueryPdu::List) {
             Ok(Query::List)
         } else {
             let mut delta = PublishDelta::default();
@@ -974,7 +974,7 @@ impl Reply {
             pdu_element.take_end(reader)?;
         }
 
-        let reply_kind = match pdus.get(0) {
+        let reply_kind = match pdus.first() {
             Some(el) => el.kind(),
             None => ReplyPduType::List
         };

diff --git a/src/crypto/keys.rs b/src/crypto/keys.rs
@@ -14,7 +14,8 @@ use ring::error::Unspecified;
 use ring::signature::VerificationAlgorithm;
 use untrusted::Input;
 use crate::oid;
-use crate::util::{base64, hex};
+#[cfg(feature = "serde")] use crate::util::base64;
+use crate::util::hex;
 use super::signature::{RpkiSignatureAlgorithm, Signature, SignatureAlgorithm};
 
 

diff --git a/src/repository/crl.rs b/src/repository/crl.rs
@@ -593,6 +593,7 @@ pub struct RevokedCertificatesIter(Captured);
 impl Iterator for RevokedCertificatesIter {
     type Item = CrlEntry;
 
+    #[allow(clippy::redundant_closure)]
     fn next(&mut self) -> Option<Self::Item> {
         self.0.decode_partial(|cons| CrlEntry::take_opt_from(cons)).unwrap()
     }

diff --git a/src/repository/manifest.rs b/src/repository/manifest.rs
@@ -42,6 +42,7 @@ pub struct Manifest {
 
 impl Manifest {
     /// Decodes a manifest from a source.
+    #[allow(clippy::redundant_closure)]
     pub fn decode<S: IntoSource>(
         source: S,
         strict: bool