Merge pull request #68 from open-quantum-safe/clippy-warnings

address two clippy warnings
open-quantum-safe · Jun 8, 2021 · 2e9a27a · 2e9a27a
2 parents c4163b4 + fa4289c
commit 2e9a27a
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/oqs/src/macros.rs b/oqs/src/macros.rs
@@ -35,8 +35,8 @@ macro_rules! newtype_buffer {
                 $name_ref { bytes }
             }
 
-            /// Copy this to the owned variant
-            pub fn to_owned(&self) -> $name {
+            /// Clone this into the owned variant
+            pub fn to_owned(self) -> $name {
                 $name {
                     bytes: self.bytes.to_vec(),
                 }
@@ -70,6 +70,7 @@ macro_rules! newtype_buffer {
 
         impl $name {
             /// Length in bytes
+            #[allow(clippy::len_without_is_empty)]
             pub fn len(&self) -> usize {
                 self.bytes.len()
             }