Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add #[must_use] to functions of slice and its iterators. #92853

Closed
wants to merge 0 commits into from
Closed

add #[must_use] to functions of slice and its iterators. #92853

wants to merge 0 commits into from

Conversation

jendrikw
Copy link
Contributor

See #89692.

@rust-highfive
Copy link
Collaborator

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @joshtriplett (or someone else) soon.

Please see the contribution instructions for more information.

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 13, 2022
@rust-log-analyzer

This comment has been minimized.

@rustbot rustbot added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label Jan 13, 2022
@rust-log-analyzer

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

@JohnCSimon JohnCSimon added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Feb 13, 2022
@pnkfelix
Copy link
Member

pnkfelix commented Mar 3, 2022

@rustbot label: +T-libs-api, -T-compiler

@rustbot rustbot added T-libs-api Relevant to the library API team, which will review and decide on the PR/issue. and removed T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Mar 3, 2022
@joshtriplett
Copy link
Member

@bors r+ rollup=iffy

(Marking rollup=iffy because changes like this often end up going through a couple of rounds of CI due to unexpected call-sites that don't get updated.)

@bors
Copy link
Contributor

bors commented Mar 3, 2022

📌 Commit cdf22c3 has been approved by joshtriplett

@bors bors 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 Mar 3, 2022
Dylan-DPC added a commit to Dylan-DPC/rust that referenced this pull request Mar 4, 2022
…lett

add #[must_use] to functions of slice and its iterators.

See rust-lang#89692.
Dylan-DPC added a commit to Dylan-DPC/rust that referenced this pull request Mar 4, 2022
…lett

add #[must_use] to functions of slice and its iterators.

See rust-lang#89692.
Dylan-DPC added a commit to Dylan-DPC/rust that referenced this pull request Mar 4, 2022
…lett

add #[must_use] to functions of slice and its iterators.

See rust-lang#89692.
Dylan-DPC added a commit to Dylan-DPC/rust that referenced this pull request Mar 4, 2022
…lett

add #[must_use] to functions of slice and its iterators.

See rust-lang#89692.
Dylan-DPC added a commit to Dylan-DPC/rust that referenced this pull request Mar 4, 2022
…lett

add #[must_use] to functions of slice and its iterators.

See rust-lang#89692.
Dylan-DPC added a commit to Dylan-DPC/rust that referenced this pull request Mar 4, 2022
…lett

add #[must_use] to functions of slice and its iterators.

See rust-lang#89692.
@bors
Copy link
Contributor

bors commented Mar 4, 2022

⌛ Testing commit cdf22c3 with merge ea25263a5702364a747107c34defc33505d67b2e...

matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Mar 4, 2022
…lett

add #[must_use] to functions of slice and its iterators.

See rust-lang#89692.
@rust-log-analyzer

This comment has been minimized.

@bors
Copy link
Contributor

bors commented Mar 4, 2022

💔 Test failed - checks-actions

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Mar 4, 2022
@bors
Copy link
Contributor

bors commented Mar 5, 2022

☔ The latest upstream changes (presumably #94628) made this pull request unmergeable. Please resolve the merge conflicts.

@jendrikw
Copy link
Contributor Author

jendrikw commented Mar 6, 2022

rebased.

@bors
Copy link
Contributor

bors commented Mar 10, 2022

☔ The latest upstream changes (presumably #94802) made this pull request unmergeable. Please resolve the merge conflicts.

@jendrikw
Copy link
Contributor Author

Sorry I messed up my branch. Re-opened at #95274.

Dylan-DPC added a commit to Dylan-DPC/rust that referenced this pull request Mar 26, 2022
add #[must_use] to functions of slice and its iterators.

Continuation of rust-lang#92853.

Tracking issue: rust-lang#89692.
bors added a commit to rust-lang-ci/rust that referenced this pull request Mar 26, 2022
add #[must_use] to functions of slice and its iterators.

Continuation of rust-lang#92853.

Tracking issue: rust-lang#89692.
flip1995 pushed a commit to flip1995/rust that referenced this pull request Apr 7, 2022
add #[must_use] to functions of slice and its iterators.

Continuation of rust-lang#92853.

Tracking issue: rust-lang#89692.
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. T-libs-api Relevant to the library API team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants