From: Nelson, E. - 2 <eri...@ba...> - 2010-01-22 19:03:48
|
Thanks for the help... It's working now, and I'm playing with it some more. I'm not sure if it matters, but in connection.hpp, in the connection constructor, there's a bit of code like try { socket_.set_option(tcp::no_delay(true)); // Don't delay writing } catch (system::system_error & e) { handler_.log(e.what()); } And the set_option call *always* seems to throw boost::asio::error::bad_descriptor. Not sure if it matters. One other thing I noticed (and don't personally care about) is the cpp-netlib coding style, like curly brace placement. I'm not sure if there are boost coding guidelines or not, or if they are enforced or not, but cpp-netlib looks a little different than other boost source code. Again, I'm perfectly happy with however it is, but if the goal is to move cpp-netlib into boost and boost cares (I don't know that either of those assumptions are true), it's probably easier to start doing it "the boost way" sooner rather than later. Thanks Erik |