-
Notifications
You must be signed in to change notification settings - Fork 903
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
eastl::ranges #465
Comments
seems an enormous task to do it alone |
I'm working on it, but there are still a lot of problems to be solved and I'm frankly don't know what I'm doing or what half of the stuff breaks. |
Just needs some problems to fix and checks whether it compiles on clang and GCC. |
there is no plans to add ranges to EASTL. EASTL has never been intended as a drop in replacement for the C++ std library. it is only intended to implement certain features - primarily the containers and algorithms. |
I'm porting range-v3 to EASTL but I'm still facing some issues that I can't seem to fix. However during my process, I found some minor issues and problems with EASTL, but I'm not confident enough to make a PR. Mainly the lack of constexpr in tuple, the lack of a constexpr addressof, a lacking "random" header, and what I think is a typo bug in heap. |
even if you aren't confident enough to submit a PR, feel free to raise issues for those things. though, the lack of random number generators is outside of the scope for EASTL. |
@jhopkins-ea I really hate to ping you again and bother you, I'm asking because if I can fix this, it could lead me to fixing views::zip and getting closer to finishing EARanges. That is the error log:
|
Are you using the most recent version of EASTL? It looks like you are running into an issue that was fixed in 3.21.12:
|
I hate my life. I should've asked for help sooner, thank you so much! |
@jhopkins-ea
|
I can't give advice on what to do with your library. You should bear in mind that eastl and specifically its algorithms are not equivalent or a conforming implementation of the standard library. I can only assume ranges-v3 relies on a conforming implementation. |
Is any plan to add ranges API to C++ ?
The text was updated successfully, but these errors were encountered: