>>> I tried updating the poll module with your patch, but it's not
>>> compiling:
>> Thanks.
> 
> Thanks for the update, tried it, it compile, but poll() will immediatly
> fail with "Invalid argument" on non connected socket. 

Have you included the other patch too?  The updated poll expects file
descriptors, not socket descriptors (and the other patch ensures that it
gets the right kind).

> I didn't tried tracing the problem yet, but if you could provide a
> complete unit-test along with the module, it would make the testing on
> different platform much easier!

You're right, there are no tests, because it's hard to write this kind
of unit test in a portable way.  Still (and unfortunately) that would
probably help for small incremental changes, but for a rewrite it is
possible that something very basic is screwed up.

Paolo


Reply via email to