logo       

Re: Missing peer_connection::setup_send() in peer_connection::on_connection: msg#00059

network.bit-torrent.libtorrent

Subject: Re: Missing peer_connection::setup_send() in peer_connection::on_connection_complete()?

yes

Arvid Norberg <arvid@xxxxxxxxx> wrote:
On Fri, Apr 28, 2006 at 07:17:32PM +0800, Tianhao Qiu wrote:
> Hi Arvid and All,
>
> I'm puzzled by some codes in peer_connection.cpp (asio branch).
>
> void peer_connection::on_connection_complete(asio::error const&)
> {
> m_connecting = false;
> }
>
> As I understand, the initiator of a connection is expected to transmit
> its handshake immediately once the connection succeeds. It looks like
> the handshake message has been added to the send buffer in the
> constructor of bt_peer_connection. There are previous calls to
> peer_connection::setup_send(), but they shall all fail since
> m_connecting = true. Therefore I expect to see calls to
> peer_connection::setup_send() immediately after m_connecting = false.
> However I didn't see any. When is setup_send() called after the
> connection is completed?

It looks like the anonymous cvs still hasn't been updated since it went
down. The peer_connection::on_connection_complete() doesn't look like that
anymore.

I guess I could put up a snapshot of the cvs. But right now I'm
experiencing strange bugs when building on winXP, so I would like to
resolve those first.

--
Arvid Norberg


-------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
Libtorrent-discuss mailing list
Libtorrent-discuss@xxxxxxxxxxxxxxxxxxxxx
https://lists.sourceforge.net/lists/listinfo/libtorrent-discuss


How low will we go? Check out Yahoo! Messenger?s low PC-to-Phone call rates.
<Prev in Thread] Current Thread [Next in Thread>
Google Custom Search

News | FAQ | advertise