Skip to content

Correct unused parameter diagnostic - #60052

Merged
bors merged 2 commits into
rust-lang:masterfrom
varkor:unused-parameter-diagnostic
Apr 19, 2019
Merged

Correct unused parameter diagnostic#60052
bors merged 2 commits into
rust-lang:masterfrom
varkor:unused-parameter-diagnostic

Conversation

@varkor

Copy link
Copy Markdown
Contributor

The message was incorrect for unused lifetime parameters. There's no need to be specific.

@rust-highfive

Copy link
Copy Markdown
Contributor

r? @estebank

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfiverust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 17, 2019
@estebank

Copy link
Copy Markdown
Contributor

r=me once travis passes

@varkor
varkorforce-pushed the unused-parameter-diagnostic branch from 842f1f4 to 048ba28CompareApril 18, 2019 17:35
@varkor

Copy link
Copy Markdown
ContributorAuthor

@bors r=estebank rollup

@bors

bors commented Apr 18, 2019

Copy link
Copy Markdown
Collaborator

📌 Commit 048ba28 has been approved by estebank

@borsbors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Apr 18, 2019
Centril added a commit to Centril/rust that referenced this pull request Apr 19, 2019
… r=estebank
Correct unused parameter diagnostic
The message was incorrect for unused lifetime parameters. There's no need to be specific.
@CentrilCentril mentioned this pull request Apr 19, 2019
Centril added a commit to Centril/rust that referenced this pull request Apr 19, 2019
… r=estebank
Correct unused parameter diagnostic
The message was incorrect for unused lifetime parameters. There's no need to be specific.
@CentrilCentril mentioned this pull request Apr 19, 2019
Centril added a commit to Centril/rust that referenced this pull request Apr 19, 2019
… r=estebank
Correct unused parameter diagnostic
The message was incorrect for unused lifetime parameters. There's no need to be specific.
@CentrilCentril mentioned this pull request Apr 19, 2019
Centril added a commit to Centril/rust that referenced this pull request Apr 19, 2019
… r=estebank
Correct unused parameter diagnostic
The message was incorrect for unused lifetime parameters. There's no need to be specific.
@CentrilCentril mentioned this pull request Apr 19, 2019
bors added a commit that referenced this pull request Apr 19, 2019
Rollup of 16 pull requests
Successful merges:
- #59915 (Implement event filtering for self-profiler.)
- #59933 (Make clear that format padding doesn't work for Debug)
- #60023 (implement specialized nth_back() for Bytes, Fuse and Enumerate)
- #60032 (Remove unwanted z-index change)
- #60041 (Simplify the returning of a Result a bit)
- #60045 (Suggest appropriate path when calling associated item on bare types)
- #60052 (Correct unused parameter diagnostic)
- #60056 (Update rustfmt to 1.2.1)
- #60060 (whitelist RTM x86 target cpu feature)
- #60061 (Change suggestion of field when not in self context)
- #60064 (Point at try `?` on errors affecting the err match arm of the desugared code)
- #60078 (Use more realistic example for thread builder)
- #60080 (Fix small errors in docs for `rchunks_exact` and `rchunks_exact_mut`.)
- #60082 (Update miri)
- #60097 (Use -mergefunc-use-aliases for any LLVM >= 8)
- #60098 (libcore: deny `elided_lifetimes_in_paths`)
Failed merges:
r? @ghost
@bors
bors merged commit 048ba28 into rust-lang:masterApr 19, 2019
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@varkor@rust-highfive@estebank@bors