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

boost::coroutines2 integration? #93

Closed
bitbugprime opened this issue Mar 12, 2018 · 4 comments
Closed

boost::coroutines2 integration? #93

bitbugprime opened this issue Mar 12, 2018 · 4 comments

Comments

@bitbugprime
Copy link

I see there's an existing pull request for this (#55), but it's been languishing in limbo for more almost six months. Is the submitted code considered incomplete or subpar or do you simply have no interest in it?

@vinniefalco
Copy link
Member

Maybe because it imposes the additional requirement of C++11, which is not currently a requirement of Boost.Asio?

@bitbugprime
Copy link
Author

I'm sure conditional compilation can solve that problem, so laggards won't hold everybody else hostage. They don't get the new feature (coroutine2 integration), but so what?

@AndrewAMD
Copy link

AndrewAMD commented Jun 26, 2018

Maybe because it imposes the additional requirement of C++11, which is not currently a requirement of Boost.Asio?

But this pull request appears to support both C++11 and pre-C++11, if I'm not mistaken.

Note: This issue title is confusing because #55 is about boost.context, not boost.coroutine2.

@ghost
Copy link

ghost commented Dec 30, 2020

This issue was moved by chriskohlhoff to chriskohlhoff/asio#603.

@ghost ghost closed this as completed Dec 30, 2020
@ghost ghost locked and limited conversation to collaborators Dec 30, 2020
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants