primal icon indicating copy to clipboard operation
primal copied to clipboard

Add "new with at least n primes" function to Sieve

Open wackywendell opened this issue 9 years ago • 4 comments

This adds a function to Sieve to create a new Sieve instance that has at least n primes in it, for a given value of n.

This also adds an implementation of Iterator and IntoIterator for Sieve, just for convenience.

As discussed in #3, the name new_at_least is pretty terrible, but I couldn't come up with anything better. I'm hoping for suggestions!

wackywendell avatar Jul 10 '15 00:07 wackywendell

r? @huonw

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

autohuonw avatar Jul 10 '15 00:07 autohuonw

Woo, thanks.

I haven't come up with any other names I like yet...

huonw avatar Jul 10 '15 16:07 huonw

Yes, neither have I. The best I've thought of is new_with_count, which does seem a bit more descriptive, but I'm not sure I like it...

wackywendell avatar Jul 10 '15 16:07 wackywendell

:umbrella: The latest upstream changes (presumably 6806007) made this pull request unmergeable. Please resolve the merge conflicts.

autohuonw avatar Oct 22 '15 01:10 autohuonw