Can this get all the way committed? The function args got changed, but the timeout still isn't actually used. Since UDP packets can go AWOL any time, you need to be able to set a timeout so you don't have your app waiting indefinately for UDP packets that will never arrive.

Begin forwarded message:

Date: Fri, 25 Nov 2005 17:16:14 -0500 From: Adam Montague amontague@siriushosting.com To: pikedevel pike-devel@lists.lysator.liu.se Subject: Protocols.SNMP.protocol()->readmsg hanging

Protocols.SNMP.protocol()->readmsg waits forever trying to read from the socket. The problem is you can send out a request on a lossy link, then try to readmsg() it, and never get an answer, hanging your script. I've attached a tiny patch to add an optional int|void timeout to readmsg().

Adam