Skip to content

Warn if no-op let _ = x statement is used #90524

Open
@sdroege

Description

@sdroege
Contributor

This is related to #90465.

The problem is code as follows

    let x = ...;
    let c = move || {
        let _ = x;
    };

or also

fn foo() {
    let x: u32;
    let _ = x; 
}

The let _ = x statement in both cases has no effect at all:

  • In the first case with the 2021 edition it does not move the x into the closure. With the 2018 edition it is moved into the closure however.
  • In the second case the variable x is not even initialized.

As the statement generally has no effect, it would be good to warn if it used like in the above cases. The author of the code probably intended something that the code is not actually doing.

Activity

RalfJung

RalfJung commented on Nov 3, 2021

@RalfJung
Member

let _ has a bunch of 'interesting' behaviors, Cc #79735 rust-lang/unsafe-code-guidelines#261

the8472

the8472 commented on Nov 5, 2021

@the8472
Member

As the statement generally has no effect, it would be good to warn if it used like in the above cases. The author of the code probably intended something that the code is not actually doing.

It's currently the recommended way to silence #[must_use] warnings.

RalfJung

RalfJung commented on Nov 5, 2021

@RalfJung
Member

It's currently the recommended way to silence #[must_use] warnings.

This issue is specifically about let _ = x where x is a variable (I think). That's different from let _ = foo() silencing a must_use on foo.

added
A-lintsArea: Lints (warnings about flaws in source code) such as unused_mut.
C-feature-requestCategory: A feature request, i.e: not implemented / a PR.
T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.
T-langRelevant to the language team
on Nov 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    A-lintsArea: Lints (warnings about flaws in source code) such as unused_mut.C-feature-requestCategory: A feature request, i.e: not implemented / a PR.T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.T-langRelevant to the language team

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

      Development

      No branches or pull requests

        Participants

        @sdroege@RalfJung@the8472@camelid

        Issue actions

          Warn if no-op `let _ = x` statement is used · Issue #90524 · rust-lang/rust