Skip to content

Make UNUSED_MUST_USE happy#7372

Closed
0xPoe wants to merge 1 commit into
rust-lang:masterfrom
0xPoe:rustin-patch-UNUSED_MUST_USE
Closed

Make UNUSED_MUST_USE happy#7372
0xPoe wants to merge 1 commit into
rust-lang:masterfrom
0xPoe:rustin-patch-UNUSED_MUST_USE

Conversation

@0xPoe

@0xPoe 0xPoe commented Jun 18, 2021

Copy link
Copy Markdown
Member

ref: rust-lang/rust#86426

Need to make UNUSED_MUST_USE check happy or the rust test will fail.

changelog: none

@rust-highfive

Copy link
Copy Markdown

r? @llogiq

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Jun 18, 2021
@flip1995

flip1995 commented Jun 18, 2021

Copy link
Copy Markdown
Member

Can you make this change directly in the rust-lang/rust PR please? That way the Rust PR doesn't have to wait for this PR to get merged and synced in 2 weeks.

@0xPoe

0xPoe commented Jun 18, 2021

Copy link
Copy Markdown
Member Author

Can you make this change directly in the rust-lang/rust PR please? That way the Rust PR doesn't have to wait for this PR to get merged and synced in 2 weeks.

Thanks! Done in rust-lang/rust@d7bb746

@0xPoe 0xPoe closed this Jun 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants