wsServer icon indicating copy to clipboard operation
wsServer copied to clipboard

Support for select() function

Open arshidkv12 opened this issue 3 years ago • 1 comments

Thank you for your good library. I think it is more helpful if you add a select function internally.

arshidkv12 avatar Feb 02 '21 08:02 arshidkv12

Hello @arshidkv12,

Thank you for your good library.

You're welcome, glad you liked it.

think it is more helpful if you add a select function internally.

Me too =). The wsServer grew from a very toy project to what is found today, there are many improvements concerning the implementation of the protocol, but something I haven't touched yet is about the client management.

As you have seen, wsServer uses threads to manage client messages, and a better approach would be the use of select() (and its siblings, like poll(), epoll()). Since I didn't expect wsServer to be used to handle a large number of clients, I have kept the thread implementation to date.

That said, adding support for this is definitely on my plans, but it is not on my priorities right now. Any contributions in this regard are welcome.

Theldus avatar Feb 03 '21 01:02 Theldus