about summary refs log tree commit diff
diff options
context:
space:
mode:
authorbors <bors@rust-lang.org>2022-03-26 20:17:04 +0000
committerbors <bors@rust-lang.org>2022-03-26 20:17:04 +0000
commita7da8a601903eb3bd70523644dd0ce936359c13a (patch)
treeb9d0fd4093df9c0c6ea64dea469ba8bddaf60ff5
parent2d85270bdf22d19072392ee15cff64de762c8315 (diff)
parent41f1413085ecf03e3a8ee33c6174d4239f62ddc0 (diff)
downloadrust-a7da8a601903eb3bd70523644dd0ce936359c13a.tar.gz
rust-a7da8a601903eb3bd70523644dd0ce936359c13a.zip
Auto merge of #95274 - jendrikw:slice-must-use, r=Dylan-DPC
add #[must_use] to functions of slice and its iterators.

Continuation of #92853.

Tracking issue: #89692.
-rw-r--r--tests/ui/bytes_nth.fixed4
-rw-r--r--tests/ui/bytes_nth.rs4
-rw-r--r--tests/ui/bytes_nth.stderr12
-rw-r--r--tests/ui/iter_next_slice.fixed8
-rw-r--r--tests/ui/iter_next_slice.rs8
-rw-r--r--tests/ui/iter_next_slice.stderr24
6 files changed, 30 insertions, 30 deletions
diff --git a/tests/ui/bytes_nth.fixed b/tests/ui/bytes_nth.fixed
index 46b7833f428..b1fb2e16bd5 100644
--- a/tests/ui/bytes_nth.fixed
+++ b/tests/ui/bytes_nth.fixed
@@ -5,7 +5,7 @@
 
 fn main() {
     let s = String::from("String");
-    s.as_bytes().get(3);
+    let _ = s.as_bytes().get(3);
     let _ = &s.as_bytes().get(3);
-    s[..].as_bytes().get(3);
+    let _ = s[..].as_bytes().get(3);
 }
diff --git a/tests/ui/bytes_nth.rs b/tests/ui/bytes_nth.rs
index c5e983d4d4e..034c54e6a42 100644
--- a/tests/ui/bytes_nth.rs
+++ b/tests/ui/bytes_nth.rs
@@ -5,7 +5,7 @@
 
 fn main() {
     let s = String::from("String");
-    s.bytes().nth(3);
+    let _ = s.bytes().nth(3);
     let _ = &s.bytes().nth(3);
-    s[..].bytes().nth(3);
+    let _ = s[..].bytes().nth(3);
 }
diff --git a/tests/ui/bytes_nth.stderr b/tests/ui/bytes_nth.stderr
index 536decf5e7f..8a7afa93450 100644
--- a/tests/ui/bytes_nth.stderr
+++ b/tests/ui/bytes_nth.stderr
@@ -1,8 +1,8 @@
 error: called `.byte().nth()` on a `String`
-  --> $DIR/bytes_nth.rs:8:5
+  --> $DIR/bytes_nth.rs:8:13
    |
-LL |     s.bytes().nth(3);
-   |     ^^^^^^^^^^^^^^^^ help: try: `s.as_bytes().get(3)`
+LL |     let _ = s.bytes().nth(3);
+   |             ^^^^^^^^^^^^^^^^ help: try: `s.as_bytes().get(3)`
    |
    = note: `-D clippy::bytes-nth` implied by `-D warnings`
 
@@ -13,10 +13,10 @@ LL |     let _ = &s.bytes().nth(3);
    |              ^^^^^^^^^^^^^^^^ help: try: `s.as_bytes().get(3)`
 
 error: called `.byte().nth()` on a `str`
-  --> $DIR/bytes_nth.rs:10:5
+  --> $DIR/bytes_nth.rs:10:13
    |
-LL |     s[..].bytes().nth(3);
-   |     ^^^^^^^^^^^^^^^^^^^^ help: try: `s[..].as_bytes().get(3)`
+LL |     let _ = s[..].bytes().nth(3);
+   |             ^^^^^^^^^^^^^^^^^^^^ help: try: `s[..].as_bytes().get(3)`
 
 error: aborting due to 3 previous errors
 
diff --git a/tests/ui/iter_next_slice.fixed b/tests/ui/iter_next_slice.fixed
index 79c1db87ac3..11ffc8edb14 100644
--- a/tests/ui/iter_next_slice.fixed
+++ b/tests/ui/iter_next_slice.fixed
@@ -6,16 +6,16 @@ fn main() {
     let s = [1, 2, 3];
     let v = vec![1, 2, 3];
 
-    s.get(0);
+    let _ = s.get(0);
     // Should be replaced by s.get(0)
 
-    s.get(2);
+    let _ = s.get(2);
     // Should be replaced by s.get(2)
 
-    v.get(5);
+    let _ = v.get(5);
     // Should be replaced by v.get(5)
 
-    v.get(0);
+    let _ = v.get(0);
     // Should be replaced by v.get(0)
 
     let o = Some(5);
diff --git a/tests/ui/iter_next_slice.rs b/tests/ui/iter_next_slice.rs
index ef9a55f3d99..e0d3aabd54a 100644
--- a/tests/ui/iter_next_slice.rs
+++ b/tests/ui/iter_next_slice.rs
@@ -6,16 +6,16 @@ fn main() {
     let s = [1, 2, 3];
     let v = vec![1, 2, 3];
 
-    s.iter().next();
+    let _ = s.iter().next();
     // Should be replaced by s.get(0)
 
-    s[2..].iter().next();
+    let _ = s[2..].iter().next();
     // Should be replaced by s.get(2)
 
-    v[5..].iter().next();
+    let _ = v[5..].iter().next();
     // Should be replaced by v.get(5)
 
-    v.iter().next();
+    let _ = v.iter().next();
     // Should be replaced by v.get(0)
 
     let o = Some(5);
diff --git a/tests/ui/iter_next_slice.stderr b/tests/ui/iter_next_slice.stderr
index 8c10a252ee0..a78d2c2d5e8 100644
--- a/tests/ui/iter_next_slice.stderr
+++ b/tests/ui/iter_next_slice.stderr
@@ -1,28 +1,28 @@
 error: using `.iter().next()` on an array
-  --> $DIR/iter_next_slice.rs:9:5
+  --> $DIR/iter_next_slice.rs:9:13
    |
-LL |     s.iter().next();
-   |     ^^^^^^^^^^^^^^^ help: try calling: `s.get(0)`
+LL |     let _ = s.iter().next();
+   |             ^^^^^^^^^^^^^^^ help: try calling: `s.get(0)`
    |
    = note: `-D clippy::iter-next-slice` implied by `-D warnings`
 
 error: using `.iter().next()` on a Slice without end index
-  --> $DIR/iter_next_slice.rs:12:5
+  --> $DIR/iter_next_slice.rs:12:13
    |
-LL |     s[2..].iter().next();
-   |     ^^^^^^^^^^^^^^^^^^^^ help: try calling: `s.get(2)`
+LL |     let _ = s[2..].iter().next();
+   |             ^^^^^^^^^^^^^^^^^^^^ help: try calling: `s.get(2)`
 
 error: using `.iter().next()` on a Slice without end index
-  --> $DIR/iter_next_slice.rs:15:5
+  --> $DIR/iter_next_slice.rs:15:13
    |
-LL |     v[5..].iter().next();
-   |     ^^^^^^^^^^^^^^^^^^^^ help: try calling: `v.get(5)`
+LL |     let _ = v[5..].iter().next();
+   |             ^^^^^^^^^^^^^^^^^^^^ help: try calling: `v.get(5)`
 
 error: using `.iter().next()` on an array
-  --> $DIR/iter_next_slice.rs:18:5
+  --> $DIR/iter_next_slice.rs:18:13
    |
-LL |     v.iter().next();
-   |     ^^^^^^^^^^^^^^^ help: try calling: `v.get(0)`
+LL |     let _ = v.iter().next();
+   |             ^^^^^^^^^^^^^^^ help: try calling: `v.get(0)`
 
 error: aborting due to 4 previous errors