If that was all we were doing, I would agree with you. But `Jeff's patch implements BOTH select and poll with an #ifdef, because not every platform has poll(). This is exactly the reason for having apr_poll(), and not using it is stupid. If the argument is performance, then back it up with numbers. If the performance is as bad as everybody seems to think it is, then fix the functions. If the problem is the API itself, then fix the API.
But the only way to fix the API is to supply it with a block of memory rather than a pool pointer.
....Roy
