Skip to content

lint iterator.map(|x| x) #5626

Closed
Closed
@matthiaskrgr

Description

@matthiaskrgr
Member

This was most likely left over after some refactoring and can be removed entirely (at least I can't think of a reason to have .map(|x| x) in the code right now..)

Activity

changed the title [-]lint .map(|x| x)[/-] [+]lint iterator.map(|x| x)[/+] on May 20, 2020
llogiq

llogiq commented on May 26, 2020

@llogiq
Contributor

Note that this will apply type adjustments on x where applicable. However, the redundant_closure lint already has a check for adjustments which can be copied to this new lint.

theo-lw

theo-lw commented on May 30, 2020

@theo-lw
Contributor

I'd like to try my hand at this!

added 2 commits that reference this issue on Jun 23, 2020

Auto merge of #5694 - wangtheo:issue-5626, r=flip1995

Auto merge of #5694 - wangtheo:issue-5626, r=matthiaskrgr

giraffate

giraffate commented on Jul 3, 2020

@giraffate
Contributor

Hi, is this issue already resolved? If so, it would be better to close this.

JohnTitor

JohnTitor commented on Jul 3, 2020

@JohnTitor
Member

Closing in favor of #5694.

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-lintArea: New lintsgood first issueThese issues are a good way to get started with Clippy

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

      Development

      No branches or pull requests

        Participants

        @matthiaskrgr@llogiq@flip1995@giraffate@JohnTitor

        Issue actions

          lint iterator.map(|x| x) · Issue #5626 · rust-lang/rust-clippy