-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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? #603
Comments
@vinniefalco commented on Mar 12, 2018, 7:10 PM UTC: Maybe because it imposes the additional requirement of C++11, which is not currently a requirement of Boost.Asio? |
@Randian commented on Mar 13, 2018, 3:47 AM UTC: 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 commented on Jun 26, 2018, 2:21 PM UTC:
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. |
Instead of boost.coroutine2 you should consider to use boost.context as replacement for boost.coroutine in boost.asio. |
@Randian commented on Mar 12, 2018, 12:56 AM UTC:
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?
This issue was moved by chriskohlhoff from boostorg/asio#93.
The text was updated successfully, but these errors were encountered: