3 files changed, 5 insertions, 5 deletions
diff --git a/openpgp/src/cert/builder.rs b/openpgp/src/cert/builder.rs
index e62bd5d0..6117a325 100644
--- a/openpgp/src/cert/builder.rs
+++ b/openpgp/src/cert/builder.rs
@@ -1006,7 +1006,7 @@ impl CertBuilder<'_> {
 
         let mut cert = Cert::try_from(packets)?;
 
-        let have_userids = self.userids.len() > 0;
+        let have_userids = !self.userids.is_empty();
 
         // Sign UserIDs.
         for (i, uid) in self.userids.into_iter().enumerate() {
diff --git a/openpgp/src/cert/bundle.rs b/openpgp/src/cert/bundle.rs
index 4c4fac2b..2b7847fe 100644
--- a/openpgp/src/cert/bundle.rs
+++ b/openpgp/src/cert/bundle.rs
@@ -623,7 +623,7 @@ impl<C> ComponentBundle<C> {
                 }
             }).collect::<Vec<&Signature>>();
 
-            if revs.len() == 0 {
+            if revs.is_empty() {
                 None
             } else {
                 Some(revs)
diff --git a/openpgp/src/cert/parser/mod.rs b/openpgp/src/cert/parser/mod.rs
index 9e888e58..31929347 100644
--- a/openpgp/src/cert/parser/mod.rs
+++ b/openpgp/src/cert/parser/mod.rs
@@ -752,7 +752,7 @@ impl<'a> CertParser<'a> {
             return Ok(None);
         }
 
-        if self.packets.len() > 0 {
+        if !self.packets.is_empty() {
             if self.packets.len() == 1 {
                 if let Err(err) = Cert::valid_start(&self.packets[0]) {
                     t!("{}", err);
@@ -927,7 +927,7 @@ impl<'a> Iterator for CertParser<'a> {
                 None => {
                     t!("EOF.");
 
-                    if self.packets.len() == 0 {
+                    if self.packets.is_empty() {
                         return None;
                     }
                     match self.cert(None) {
@@ -968,7 +968,7 @@ impl<'a> Iterator for CertParser<'a> {
                             self.saw_error = true;
                             return Some(Err(err));
                         }
-                        None if self.packets.len() == 0 => {
+                        None if self.packets.is_empty() => {
                             t!("Packet iterator was empty");
                             Ok(None)
                         }