openpgp: Pass MessageStructure by value, not reference.

  - Instead of passing MessageStructure to VerificationHelper::check
    by reference, pass it by value.

  - After calling VerificationHelper::check, it is dropped.  Passing
    it by value allows the caller to avoid some cloning.

1 files changed, 3 insertions, 3 deletions

diff --git a/openpgp/src/serialize/stream.rs b/openpgp/src/serialize/stream.rs
index cabe130b..9c5977c9 100644
--- a/openpgp/src/serialize/stream.rs
+++ b/openpgp/src/serialize/stream.rs
@@ -252,7 +252,7 @@ impl<'a> Signer<'a> {
     ///         Ok(vec![self.0.clone()])
     ///     }
     ///
-    ///     fn check(&mut self, structure: &MessageStructure)
+    ///     fn check(&mut self, structure: MessageStructure)
     ///              -> openpgp::Result<()> {
     ///         if let MessageLayer::SignatureGroup { ref results } =
     ///             structure.iter().nth(0).unwrap()
@@ -359,7 +359,7 @@ impl<'a> Signer<'a> {
     ///         Ok(vec![self.0.clone()])
     ///     }
     ///
-    ///     fn check(&mut self, structure: &MessageStructure)
+    ///     fn check(&mut self, structure: MessageStructure)
     ///              -> openpgp::Result<()> {
     ///         if let MessageLayer::SignatureGroup { ref results } =
     ///             structure.iter().nth(0).unwrap()
@@ -1665,7 +1665,7 @@ mod test {
                                -> Result<Vec<Cert>> {
                 Ok(Vec::new())
             }
-            fn check(&mut self, _structure: &MessageStructure) -> Result<()> {
+            fn check(&mut self, _structure: MessageStructure) -> Result<()> {
                 Ok(())
             }
         }