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

impl ParallelIterator for RangeInclusive #647

Merged
merged 1 commit into from
May 17, 2019
Merged

Conversation

cuviper
Copy link
Member

@cuviper cuviper commented Mar 26, 2019

All the grunt work is forwarded to range::Iter, with fixups for the
end case as needed.

Fixes #572.

All the grunt work is forwarded to `range::Iter`, with fixups for the
end case as needed.
@cuviper
Copy link
Member Author

cuviper commented May 8, 2019

I've rebased this for 1.26 after #656 -- in particular, we don't need build-script detection anymore.

if let Some(end) = end.checked_add(1) {
(start..end).into_par_iter().$method($( $arg ),*)
} else {
(start..end).into_par_iter().chain(once(end)).$method($( $arg ),*)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cute

@nikomatsakis
Copy link
Member

bors r+

bors bot added a commit that referenced this pull request May 17, 2019
647: impl ParallelIterator for RangeInclusive r=nikomatsakis a=cuviper

All the grunt work is forwarded to `range::Iter`, with fixups for the
end case as needed.

Fixes #572.

Co-authored-by: Josh Stone <[email protected]>
@bors
Copy link
Contributor

bors bot commented May 17, 2019

Build succeeded

@bors bors bot merged commit a6f40b3 into rayon-rs:master May 17, 2019
@sunjay
Copy link

sunjay commented May 24, 2019

Any chance of releasing a new version of rayon so this feature can be used?

@cuviper
Copy link
Member Author

cuviper commented Jun 6, 2019

#661 is the release PR.

@cuviper cuviper deleted the range-inc branch March 11, 2020 23:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for RangeInclusive
3 participants