search

Ernillew / wl500g

Automatically exported from code.google.com/p/wl500g
0 stars 0 forks source link

p910nd enhancements #187

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
Would be great if you apply fixes proposed by Mustafa Karaca on 
http://p910nd.sourceforge.net/:
It appears that select() is called with invalid timeout value. You can find the 
.diff filed as attached.
Also daemon blocks on select() for a long time if the network connection is 
dropped (for example, if we unplug the network cable when a data transfer is 
happening from a windows xp box). So, specifying a timeout value for the 
network connection could be a solution (see p910d_timeout.diff). If the 
connection is dropped because of some unknown problem, daemon will close the 
current stream after specified amount of time has passed without a network 
activity and will start listening for new connections to arrive.

diff files are available on p910nd site.

I understand that it's still not in the mainstream, but p910nd isn't actively 
developing right now, but features are quite hot.

Original issue reported on code.google.com by dusha...@gmail.com on 12 Dec 2010 at 10:20

GoogleCodeExporter commented 9 years ago 
[deleted comment]
GoogleCodeExporter commented 9 years ago 
If you took a look at sources, p910d.diff already included for months.

p910d_timeout.diff will be included.

Original comment by lly.dev on 13 Dec 2010 at 8:21

GoogleCodeExporter commented 9 years ago 
Thanks a lot. p910d_timeout.diff is needed in case of connection drop during 
printing, after this customer have to restart p910d daemon and reattach the 
cable or just reboot the router, which doesn't look like a good solution :)

Will you add some field to modify timeout from web ui as well?

Original comment by dusha...@gmail.com on 13 Dec 2010 at 1:21

GoogleCodeExporter commented 9 years ago 
Patch added in r2425.

To modify webUI I need reasonable default value, can you suggest it?

Original comment by lly.dev on 16 Dec 2010 at 5:32

GoogleCodeExporter commented 9 years ago 
Hm, for compatibility purposes i'd leave it non-specified by default (turned 
off, unchecked, etc.), so it'll work as original product.

Original comment by dusha...@gmail.com on 17 Dec 2010 at 11:11

GoogleCodeExporter commented 9 years ago

Original comment by lly.dev on 6 Mar 2011 at 12:31