Re: cvs socket extension

On Wed, Jul 25, 2001 at 06:37:25PM -0700, Chris Vandomelen wrote : > > At 20:15 24.07.2001 +0200, Markus Fischer wrote: > > >I've come up with a patch based on the ideas and suggestion that > > >came up in this thread: > > > > > >- Let the fd_set resource keep track of the highest socket; nuked > > > _select() first parameter and determine it ourself > > >- Swapped _set(), _clear() and _isset() parameters and > > > recognize plain sockets as well as array of sockets > > > > > >The patch is against current CVS and only compiled unter linux > > >(debian unstable) so far. > > > > > >Please everyone interested test and verify it and let me know > > >what you think. Due my lack of karma I can't commit it anyway, > > >feel free to do so if you find it appropriate. > > > > please let me know wether the attached patch fixes your concerns. > > Looks good. But it doesn't matter if someone passes no sets into select(): > if there are no sets, it essentially becomes a usleep(). Yes I know, I had another patch which took care of this. But Daniels latest commit took care of everything, looks very good now :) - Markus