Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IOCP in Windows #24

Closed
Cheedoong opened this Issue Oct 31, 2016 · 6 comments

Comments

Projects
None yet
3 participants
@Cheedoong
Copy link

Cheedoong commented Oct 31, 2016

One drawback of glib is that it does not support IOCP in Windows.
I hope you will consider about implementing it.

@waruqi

This comment has been minimized.

Copy link
Member

waruqi commented Nov 1, 2016

We will support iocp in the future. : )

@TitanSnow

This comment has been minimized.

Copy link

TitanSnow commented Mar 27, 2017

It seems that IOCP is only supported on Windows. I think to add it may reduce compatibility。

Correspondingly, there's epoll in Linux, /dev/poll in Solaris. They're all vary.

I think a new, unified, abstract, high-level and fast network model may be a better choice but it may be a hard work.

@waruqi

This comment has been minimized.

Copy link
Member

waruqi commented Mar 28, 2017

@TitanSnow tbox provide a unified and abstract poller interface to wrap epoll, poll, select and kqueue.

And I will continue to support iocp in future, but it may take some time.

@TitanSnow

This comment has been minimized.

Copy link

TitanSnow commented Mar 28, 2017

That sounds great👍

@waruqi waruqi added todo and removed todo labels Jun 2, 2017

@waruqi waruqi added this to the todo milestone Jun 3, 2017

@waruqi waruqi added the in progress label Feb 7, 2018

@waruqi

This comment has been minimized.

Copy link
Member

waruqi commented Jul 6, 2018

@Cheedoong I've already supported IOCP for coroutine and poller on windows. You can download master version and try it.

@waruqi waruqi modified the milestones: todo, v1.6.3 Jul 6, 2018

@waruqi waruqi added done and removed in progress labels Jul 6, 2018

@Cheedoong

This comment has been minimized.

Copy link
Author

Cheedoong commented Jul 12, 2018

Great job! We will try it.

@waruqi waruqi closed this Aug 15, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.