-
Notifications
You must be signed in to change notification settings - Fork 118
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
Update range() for additional features in python's range #55
Comments
Second Part of #55. Not supporting floating point types. I can't yet get the float version perfect.
Would it be against the philosophy of this project to extend It would be nice to get log time |
There are aspects of random access iterators that can't be easily handled by range, since there isn't an underlying sequence to point to. Even having it as a ForwardIterator is sort of lying. This is not to say that you couldn't go about implementing some of the operators, but standard libraries will still use the iterator category to figure it out, and you can't update the category without implementing all of the required functions. Look at libc++'s implementation of I've thought about reaching for more advanced iterator tags before with range, I think bidirectional is doable, but the situation is more complicated with random access. All that said, if you would like to look for a path forward I'm happy to talk or review PRs. |
Hmm, I think the only real difficulty (though it may be insurmountable) is that the dereference operations are meant to return references, not values. All of the other operations look pretty simple. Subclassing the trait tag with that imperfection is probably a dreadful idea, though, so I'll go another few years without a lightweight, well-maintained way to binary search over integers :-/. Thanks for the help finding the flaw in that plan. |
Is there a std::ranges solution that you're alluding to coming in a few
years?
…On Sun, Jul 21, 2019, 5:31 PM Matthew Steel ***@***.***> wrote:
Hmm, I think the only real difficulty (though it may be insurmountable) is
that the dereference operations are meant to return references, not values.
All of the other operations look pretty simple.
Subclassing the trait tag with that imperfection is probably a dreadful
idea, though, so I'll go another few years without a lightweight way to
binary search over integers :-/.
Thanks for the help finding the flaw in that plan.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<#55?email_source=notifications&email_token=AAIWWWNWWJPOYYOJQPPFWPTQAT5WTA5CNFSM4HARYGNKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2OPA2A#issuecomment-513601640>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAIWWWMH2MTCQXEDMLBVGJTQAT5WTANCNFSM4HARYGNA>
.
|
I wouldnt know sorry, I was just voicing a vague hope that enough people will reinvent the wheel that one of them will turn out to be useful. |
Give a range object
r
r.start()
,r.stop()
, andr.stop()
constexpr member functions to match the.start
,.stop
, and.step
properties.r.size()
andr.length()
to matchlen(r)
r[index]
The text was updated successfully, but these errors were encountered: