Re: [boost] [review][beast] Review of Beast starts today : July 1 - July 10

2017-07-09 20:22 GMT+02:00 Vinnie Falco via Boost :

On Sun, Jul 9, 2017 at 11:02 AM, Oliver Kowalke via Boost wrote:

...

warnings can be disabled

I do disable them but it doesn't help: <https://github.com/vinniefalco/Beast/blob/1bc30cb6e4ce0fd2e288b2cd74d18f 857801cbc7/Jamroot#L82>

...

or boost.coroutien2 (C++11) could be used instead

The problem is that Beast doesn't use Boost.Coroutine. It is Asio that uses Boost.Coroutine. Beast just uses boost::asio::spawn which launches a coroutine using Boost.Coroutine.

in order to use Boost.Coroutine2 (and I think this is a good idea), Boost.Asio has to be patched to use it. The problem is this will break everyone using Asio coroutines, as Coroutine2 is 1. not compatible with the Coroutine interface, and 2. not available in C++11.

I've had already provided a pull-request: https://github.com/boostorg/asio/pull/52