Skip to content

Tracking issue for RFC#1685: Deprecate anonymous parameters  #41686

Closed
@aturon

Description

@aturon
Member

This is a tracking issue for the RFC "Deprecate anonymous parameters " (rust-lang/rfcs#1685).

Steps:

  • Implement the RFC (cc @rust-lang/compiler -- can anyone write up mentoring instructions?)
    Adjust documentation (see instructions on forge)

Unresolved questions:

  • The precise deprecation strategy

Activity

added
B-RFC-approvedBlocker: Approved by a merged RFC but not yet implemented.
T-langRelevant to the language team
on May 1, 2017
est31

est31 commented on May 1, 2017

@est31
Member

I could work on this. From reading the RFC discussion, I think the "soft" deprecation strategy has been chosen? So on the compiler side, we'll need an allow by default (deprecation) lint for anonymous parameters, right?

aturon

aturon commented on May 1, 2017

@aturon
MemberAuthor

@est31 Awesome! And yes, that's correct. This will be most effective if we add the lint to Clippy as well.

(The open question is around the criteria for moving to a warn-by-default strategy in the future.)

added a commit that references this issue on May 2, 2017
est31

est31 commented on May 2, 2017

@est31
Member

About the clippy lint, I'll leave that to someone else. Given #41692, it should be mostly copy paste, but maybe with official rust claiming the lint name one has to chose another one. No idea.

added 2 commits that reference this issue on May 2, 2017

Rollup merge of rust-lang#41692 - est31:anon_params, r=eddyb

Rollup merge of rust-lang#41693 - est31:anon_params_removal, r=eddyb

matklad

matklad commented on May 5, 2017

@matklad
Member

IJR inspection: intellij-rust/intellij-rust#1211

I'll try to add an automatic fix to rustfmt as well. Not sure it's a great idea, but I want to try :)

added
C-tracking-issueCategory: An issue tracking the progress of sth. like the implementation of an RFC
on Jul 27, 2017

65 remaining items

Loading
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

    B-RFC-approvedBlocker: Approved by a merged RFC but not yet implemented.C-tracking-issueCategory: An issue tracking the progress of sth. like the implementation of an RFCT-langRelevant to the language teamdisposition-mergeThis issue / PR is in PFCP or FCP with a disposition to merge it.finished-final-comment-periodThe final comment period is finished for this PR / Issue.

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

      Development

      No branches or pull requests

        Participants

        @steveklabnik@nikomatsakis@vi@SimonSapin@aturon

        Issue actions

          Tracking issue for RFC#1685: Deprecate anonymous parameters · Issue #41686 · rust-lang/rust