Re: [Boost-users] Problem finally found!

Subject: Re: [Boost-users] Problem finally found!
From: ingo_at_[hidden]
Date: 2013-01-06 04:47:08

Hi Juan,

I wasn't asked, but it looks like some kind of MS Visio. Visio is fine for small
diagrams, but it's a nightmare for bigger ones because of the over-smart
connection arrows... For bigger things I'm using EnterpriseArchitect.

Cheers,

Ingo

"Juan Ramírez" <juanantonio.ram_at_[hidden]> hat am 6. Januar 2013 um 03:55
geschrieben:

> It's fine to call start_accept() inside the handle_accept() function, if you
> don't do it your server will not be able to accept another client
> 
>  You should check if the ioservice has enough threads to handle more than one
> client, maybe you have a single thread being blocked by the call to
>  session->start(), if this is the case, your server cannot accept another
> client until the thread is released (make sure the ioservice has an
> oioservice::work object attached, btw).
> 
>  __________
>  Saludos!
>       Juan
> 
> 
> 
>  On Fri, Jan 4, 2013 at 2:30 PM, Gonzalo Garramuno <ggarra13_at_[hidden]
> <mailto:ggarra13_at_[hidden]> > wrote:
>    > > My problem was that acceptor_ was called twice and two sessions were
>    > > created.  The first session was used properly while the second session
>    > > missed the socket setup.
> > 
> >    class server
> >    {
> >    public:
> >      server(boost::asio::io_ service& io_service,
> >         const tcp::endpoint& listen_endpoint,
> >         mrv::ViewerUI* v)
> >        : io_service_(io_service),
> >          acceptor_(io_service, listen_endpoint),
> >          ui_( v )
> >      {
> >         start_accept();
> >      }
> > 
> >      void start_accept()
> >      {
> >         tcp_session_ptr new_session(
> >                     boost::make_shared<tcp_ session>(
> >                                     boost::ref(
> >                                            io_service_
> >                                            ),
> >                                     boost::ref(ui_)
> >                                     )
> >                     );
> > 
> > 
> >         std::cerr << "******** TCP SESSION " << new_session << std::endl;
> > 
> >        acceptor_.async_accept(new_ session->socket(),
> >                   boost::bind(&server::handle_ accept, this,
> >                           new_session, _1));
> >      }
> > 
> >      void handle_accept(tcp_session_ptr session,
> >                 const boost::system::error_code& ec)
> >      {
> >        if (!ec)
> >        {
> >          session->start();
> >        }
> >        start_accept(); // BAD
> >      }
> > 
> >    Changed to:
> > 
> >      void handle_accept(tcp_session_ptr session,
> >                 const boost::system::error_code& ec)
> >      {
> >        if (!ec)
> >        {
> >          session->start();
> >        }
> >        else
> >        {
> >          start_accept();
> >        }
> >      }
> > 
> > 
> >    I still don't know why this is so and the problem is not fully resolved,
> > as only the server change in the timeline works only once, but at least the
> > server now works.
> >    ______________________________ _________________
> >    Boost-users mailing list
> >    Boost-users_at_[hidden] <mailto:Boost-users_at_[hidden]>
> >    http://lists.boost.org/mailman/listinfo.cgi/boost-users
> > <http://lists.boost.org/mailman/listinfo.cgi/boost-users>
> >  >  _______________________________________________
>  Boost-users mailing list
>  Boost-users_at_[hidden]
>  http://lists.boost.org/mailman/listinfo.cgi/boost-users
>