Skip to content

vec::find should return a reference #3793

Closed
@erickt

Description

@erickt
Contributor

Either that or we add a vec::find_ref function. Right now it returns a copy, so it trips up the no-implicit-copies warning.

Activity

catamorphism

catamorphism commented on Mar 20, 2013

@catamorphism
Contributor

Not critical for 0.6, de-milestoning

thestinger

thestinger commented on Mar 20, 2013

@thestinger
Contributor

I don't think vectors should expose this method at all, it should just be an iter module function for any iterator. The vector can't do anything special to make it faster.

thestinger

thestinger commented on Jun 12, 2013

@thestinger
Contributor

Closing, since iterators make methods like this obsolete.

added a commit that references this issue on Aug 10, 2024

Auto merge of rust-lang#3793 - rust-lang:rustup-2024-08-07, r=RalfJung

added a commit that references this issue on Aug 18, 2024

Auto merge of rust-lang#3793 - rust-lang:rustup-2024-08-07, r=RalfJung

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

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

      Development

      No branches or pull requests

        Participants

        @erickt@catamorphism@thestinger

        Issue actions

          vec::find should return a reference · Issue #3793 · rust-lang/rust