indexing: reword help

Author: hkBst

compiler/rustc_error_codes/src/error_codes/E0608.md

@@ -1,5 +1,4 @@
-An attempt to use index on a type which doesn't implement the `std::ops::Index`
-trait was performed.
+Indexing was attempted on a type which doesn't implement the `std::ops::Index` trait.
 
 Erroneous code example:
 
@@ -16,3 +15,6 @@ let v: Vec<u8> = vec![0, 1, 2, 3];
 // The `Vec` type implements the `Index` trait so you can do:
 println!("{}", v[2]);
 ```
+
+Tuples and structs are indexed with dot (`._`), not with brackets (`[_]`).
+Tuple element names are their positions: tuple = (tuple.0, tuple.1, ...)

compiler/rustc_hir_typeck/src/expr.rs

@@ -3552,35 +3552,21 @@ impl<'a, 'tcx> FnCtxt<'a, 'tcx> {
                     );
                     // Try to give some advice about indexing tuples.
                     if let ty::Tuple(types) = base_t.kind() {
-                        let mut needs_note = true;
-                        // If the index is an integer, we can show the actual
-                        // fixed expression:
+                        err.help(
+                            "tuples are indexed (like structs) with dot (`._`) not with brackets (`[_]`), \
+                             element names are their positions: tuple = (tuple.0, tuple.1, ...)",
+                        );
+                        // If index is an unsuffixed integer, show the fixed expression:
                         if let ExprKind::Lit(lit) = idx.kind
                             && let ast::LitKind::Int(i, ast::LitIntType::Unsuffixed) = lit.node
-                            && i.get()
-                                < types
-                                    .len()
-                                    .try_into()
-                                    .expect("expected tuple index to be < usize length")
+                            && i.get() < types.len().try_into().expect("tuple length fits in u128")
                         {
                             err.span_suggestion(
                                 brackets_span,
-                                "to access tuple elements, use",
+                                format!("to access tuple element `{i}`, use"),
                                 format!(".{i}"),
                                 Applicability::MachineApplicable,
                             );
-                            needs_note = false;
-                        } else if let ExprKind::Path(..) = idx.peel_borrows().kind {
-                            err.span_label(
-                                idx.span,
-                                "cannot access tuple elements at a variable index",
-                            );
-                        }
-                        if needs_note {
-                            err.help(
-                                "to access tuple elements, use tuple indexing \
-                                        syntax (e.g., `tuple.0`)",
-                            );
                         }
                     }
 

tests/ui/indexing/index_message.stderr

@@ -2,7 +2,9 @@ error[E0608]: cannot index into a value of type `({integer},)`
   --> $DIR/index_message.rs:3:14
    |
 LL |     let _ = z[0];
-   |              ^^^ help: to access tuple elements, use: `.0`
+   |              ^^^ help: to access tuple element `0`, use: `.0`
+   |
+   = help: tuples are indexed (like structs) with dot (`._`) not with brackets (`[_]`), element names are their positions: tuple = (tuple.0, tuple.1, ...)
 
 error: aborting due to 1 previous error
 

tests/ui/issues/issue-27842.stderr

@@ -2,25 +2,25 @@ error[E0608]: cannot index into a value of type `({integer}, {integer}, {integer
   --> $DIR/issue-27842.rs:4:16
    |
 LL |     let _ = tup[0];
-   |                ^^^ help: to access tuple elements, use: `.0`
+   |                ^^^ help: to access tuple element `0`, use: `.0`
+   |
+   = help: tuples are indexed (like structs) with dot (`._`) not with brackets (`[_]`), element names are their positions: tuple = (tuple.0, tuple.1, ...)
 
 error[E0608]: cannot index into a value of type `({integer}, {integer}, {integer})`
   --> $DIR/issue-27842.rs:9:16
    |
 LL |     let _ = tup[i];
-   |                ^-^
-   |                 |
-   |                 cannot access tuple elements at a variable index
+   |                ^^^
    |
-   = help: to access tuple elements, use tuple indexing syntax (e.g., `tuple.0`)
+   = help: tuples are indexed (like structs) with dot (`._`) not with brackets (`[_]`), element names are their positions: tuple = (tuple.0, tuple.1, ...)
 
 error[E0608]: cannot index into a value of type `({integer},)`
   --> $DIR/issue-27842.rs:14:16
    |
 LL |     let _ = tup[3];
    |                ^^^
    |
-   = help: to access tuple elements, use tuple indexing syntax (e.g., `tuple.0`)
+   = help: tuples are indexed (like structs) with dot (`._`) not with brackets (`[_]`), element names are their positions: tuple = (tuple.0, tuple.1, ...)
 
 error: aborting due to 3 previous errors
 

tests/ui/span/suggestion-non-ascii.stderr

@@ -2,7 +2,9 @@ error[E0608]: cannot index into a value of type `({integer},)`
   --> $DIR/suggestion-non-ascii.rs:3:24
    |
 LL |     println!("☃{}", tup[0]);
-   |                        ^^^ help: to access tuple elements, use: `.0`
+   |                        ^^^ help: to access tuple element `0`, use: `.0`
+   |
+   = help: tuples are indexed (like structs) with dot (`._`) not with brackets (`[_]`), element names are their positions: tuple = (tuple.0, tuple.1, ...)
 
 error: aborting due to 1 previous error
 

tests/ui/typeck/coercion-check-for-indexing-expression-issue-40861.stderr

@@ -4,7 +4,7 @@ error[E0608]: cannot index into a value of type `()`
 LL |     ()[f(&[1.0])];
    |       ^^^^^^^^^^^
    |
-   = help: to access tuple elements, use tuple indexing syntax (e.g., `tuple.0`)
+   = help: tuples are indexed (like structs) with dot (`._`) not with brackets (`[_]`), element names are their positions: tuple = (tuple.0, tuple.1, ...)
 
 error: aborting due to 1 previous error