Skip to content

Update Condvar docs to use match ergonomics #62857

Description

@KevinWMatthews

In std::sync::Condvar's docs, it looks like line 39:

let &(ref lock, ref cvar) = &*pair2;

and line 51

let &(ref lock, ref cvar) = &*pair;

were written before the match ergonomics RFC was merged.

Is it reasonable/preferable to change these lines to leverage match ergonomics? They would become:

let (lock, cvar) = &*pair2;

and

let (lock, cvar) = &*pair

respectively.

Activity

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-docsArea: Documentation for any part of the project, including the compiler, standard library, and toolsC-enhancementCategory: An issue proposing an enhancement or a PR with one.E-easyCall for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.T-libs-api[DEPRECATED; DO NOT USE]

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions