Skip to content

Update Condvar docs to use match ergonomics #62857

Closed
@KevinWMatthews

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.

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-apiRelevant to the library API team, which will review and decide on the PR/issue.

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions