Skip to content

Rollup of 3 pull requests #5666

New issue

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

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

Already on GitHub? # to your account

Merged
merged 6 commits into from
May 31, 2020
Merged

Rollup of 3 pull requests #5666

merged 6 commits into from
May 31, 2020

Conversation

flip1995
Copy link
Member

@flip1995 flip1995 commented May 31, 2020

Successful merges:

Failed merges:

r? @ghost

changelog: rollup

ebroto and others added 6 commits May 28, 2020 00:58
… r=yaahc,flip1995

new lint: vec_resize_to_zero

implements rust-lang#5444

changelog: new lint vec_resize_to_zero
len_zero: skip ranges if feature `range_is_empty` is not enabled

If the feature is not enabled, calling `is_empty()` on a range is ambiguous. Moreover, the two possible resolutions are unstable methods, one inherent to the range and the other being part of the `ExactSizeIterator` trait.

Since `len_zero` only checks for existing `is_empty()` inherent methods, we only take into account the `range_is_empty` feature.

Related: rust-lang/rust#48111 (comment)

changelog: len_zero: avoid linting ranges without #![feature(range_is_empty)]

Fixes: rust-lang#3807
…shearth

add testcase that no longer ICEs

Fixes rust-lang#3969

changelog: none
@flip1995
Copy link
Member Author

@bors r+

@bors
Copy link
Contributor

bors commented May 31, 2020

📌 Commit 00c656d has been approved by flip1995

@flip1995
Copy link
Member Author

@bors p=3

@bors
Copy link
Contributor

bors commented May 31, 2020

⌛ Testing commit 00c656d with merge a98a3a1...

bors added a commit that referenced this pull request May 31, 2020
Rollup of 3 pull requests

Successful merges:

 - #5637 (new lint: vec_resize_to_zero)
 - #5656 (len_zero: skip ranges if feature `range_is_empty` is not enabled)
 - #5663 (add testcase that no longer ICEs)

Failed merges:

r? @ghost

changelog: rollup
@bors
Copy link
Contributor

bors commented May 31, 2020

💔 Test failed - checks-action_test

@flip1995
Copy link
Member Author

@bors retry

@bors
Copy link
Contributor

bors commented May 31, 2020

⌛ Testing commit 00c656d with merge 8b191b5...

@bors
Copy link
Contributor

bors commented May 31, 2020

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: flip1995
Pushing 8b191b5 to master...

@bors bors merged commit 8b191b5 into rust-lang:master May 31, 2020
@flip1995 flip1995 deleted the rollup-yjyvvbg branch May 31, 2020 19:10
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants