Re: [bme-develop] WatchNetEndpoint

[Simon T. <sim...@ga...>]

> > Hi all,
> Hello
> 
> > I've done a bit of integration of my WatchNetEndpoint class with 
> > bme. 
> > It's shown up the fact that the protocol code (in 
> > MsnServerHandler::receive) only gets 1 byte at a time from the 
> > BNetEndpoint. I think this should be changed as it is not very 
> > efficient to keep calling the net_server like this; and it may 
> > explain 
> > some of the instability experienced by people with fast 
> > connections. 
> > What does anybody think?
> > 
> You're right, it is really inefficient to have so many receive calls, 
> it's been something I always wanted to change, but have never put 
> much 
> effort into it, mainly because the MsnServerHandler::receive method 
> should copy only one Msn command to the buffer. I never came up with 
> a 
> good solution to this problem, only the easy but inefficient way we 
> have now. Do you have any suggestions for this?

I read something about it in the hypothetic.org FAQ:
http://www.hypothetic.org/docs/msn/resources/faq.php

I'll have a go at something like this.

> > Also, because my WatchNetEndpoint class adds a "received" item 
> > after 
> > each call to BNetEndpoint::Receive, the current code creates a very 
> > messy-looking log:
> > 
> > http://si-msn.port5.com/netwatch.png
> > 
> It looks good!!, that will help me sending the P2P messages. 

I've not added the hex support yet, I'll fix the server handlers to get 
a larger buffer first, then look at that side of things.

> Daniel.

Simon