Merge 62bc31b into ad232bf

jbaublitz · Nov 16, 2019 · 47d839e · 47d839e
2 parents ad232bf + 62bc31b
commit 47d839e
Show file tree

Hide file tree

Showing 5 changed files with 17 additions and 6 deletions.
diff --git a/README.md b/README.md
@@ -54,6 +54,7 @@ where
     /// Doc comments are supported!
     /// Multiline, even.
     #[inline(always)]
+    #[must_use]
     fn private(&self) -> &T {
         &self.private
     }
@@ -84,12 +85,14 @@ where
     /// Doc comments are supported!
     /// Multiline, even.
     #[inline(always)]
+    #[must_use]
     fn private_mut(&mut self) -> &mut T {
         &mut self.private
     }
     /// Doc comments are supported!
     /// Multiline, even.
     #[inline(always)]
+    #[must_use]
     pub fn public_mut(&mut self) -> &mut T {
         &mut self.public
     }
@@ -101,6 +104,7 @@ where
     /// Doc comments are supported!
     /// Multiline, even.
     #[inline(always)]
+    #[must_use]
     pub fn public(&self) -> T {
         self.public
     }

diff --git a/src/generate.rs b/src/generate.rs
@@ -125,6 +125,7 @@ pub fn implement(field: &Field, mode: &GenMode, params: &GenParams) -> TokenStre
                 quote! {
                     #(#doc)*
                     #[inline(always)]
+                    #[must_use]
                     #visibility fn #fn_name(&self) -> &#ty {
                         &self.#field_name
                     }
@@ -134,6 +135,7 @@ pub fn implement(field: &Field, mode: &GenMode, params: &GenParams) -> TokenStre
                 quote! {
                     #(#doc)*
                     #[inline(always)]
+                    #[must_use]
                     #visibility fn #fn_name(&self) -> #ty {
                         self.#field_name
                     }
@@ -153,6 +155,7 @@ pub fn implement(field: &Field, mode: &GenMode, params: &GenParams) -> TokenStre
                 quote! {
                     #(#doc)*
                     #[inline(always)]
+                    #[must_use]
                     #visibility fn #fn_name(&mut self) -> &mut #ty {
                         &mut self.#field_name
                     }

diff --git a/src/lib.rs b/src/lib.rs
@@ -48,6 +48,7 @@ where
     /// Doc comments are supported!
     /// Multiline, even.
     #[inline(always)]
+    #[must_use]
     fn private(&self) -> &T {
         &self.private
     }
@@ -78,12 +79,14 @@ where
     /// Doc comments are supported!
     /// Multiline, even.
     #[inline(always)]
+    #[must_use]
     fn private_mut(&mut self) -> &mut T {
         &mut self.private
     }
     /// Doc comments are supported!
     /// Multiline, even.
     #[inline(always)]
+    #[must_use]
     pub fn public_mut(&mut self) -> &mut T {
         &mut self.public
     }
@@ -95,6 +98,7 @@ where
     /// Doc comments are supported!
     /// Multiline, even.
     #[inline(always)]
+    #[must_use]
     pub fn public(&self) -> T {
         self.public
     }

diff --git a/tests/copy_getters.rs b/tests/copy_getters.rs
@@ -101,19 +101,19 @@ mod submodule {
         #[test]
         fn test_plain() {
             let val = Plain::default();
-            val.private_accessible();
+            let _ = val.private_accessible();
         }
 
         #[test]
         fn test_generic() {
             let val = Generic::<usize>::default();
-            val.private_accessible();
+            let _ = val.private_accessible();
         }
 
         #[test]
         fn test_where() {
             let val = Where::<usize>::default();
-            val.private_accessible();
+            let _ = val.private_accessible();
         }
 
         #[test]

diff --git a/tests/getters.rs b/tests/getters.rs
@@ -101,19 +101,19 @@ mod submodule {
         #[test]
         fn test_plain() {
             let val = Plain::default();
-            val.private_accessible();
+            let _ = val.private_accessible();
         }
 
         #[test]
         fn test_generic() {
             let val = Generic::<usize>::default();
-            val.private_accessible();
+            let _ = val.private_accessible();
         }
 
         #[test]
         fn test_where() {
             let val = Where::<usize>::default();
-            val.private_accessible();
+            let _ = val.private_accessible();
         }
 
         #[test]