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

(asys) Add libuv wrappers #306

Draft
wants to merge 22 commits into
base: master
Choose a base branch
from

Conversation

Aurel300
Copy link
Member

@Aurel300 Aurel300 commented Sep 16, 2019

See HaxeFoundation/haxe#8817

Some TODOs marked in code. Additional things to investigate:

  • use GC finalisers rather than noptr? (e.g. close file when handle is no longer reachable from Haxe code)

CI TODO:

  • build 32-bit libuv on Linux32
  • install libuv on Windows

@Aurel300 Aurel300 mentioned this pull request Sep 16, 2019
4 tasks
@ConstNW
Copy link
Contributor

ConstNW commented Nov 26, 2019

@Aurel300, do you have any progress on this? can i take a look?

@sonygod
Copy link

sonygod commented Jan 27, 2020

any fixed here ? @Aurel300 ?

I like your udp function in libuv.

@Aurel300
Copy link
Member Author

@ConstNW @sonygod This is temporarily on hold. First we need to finalise the actual asys APIs (HaxeFoundation/haxe#8832) – we have agreed that some changes need to be made (e.g. dropping the streams, replacing the socket data signals with callback-based methods for simplicity and consistency). I believe @RealyUniqueName is currently working on that. Once the APIs are decided, individual targets can be implemented.

@piboistudios
Copy link

@Aurel300 Wow, I've been researching async IO and brainstorming solutions to implementing it in Haxe (particularly HL) and I've repeatedly stumbled across your name.

Glad to see you are diligently working at this task, I wish I could be of help, but I haven't compiled C in over 2 years, all I can say is I'm sure the entire Haxe community looks forward to your success! This is the most exciting proposal I've read about in a while.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants