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

Add Eio.Net.getaddrinfo_{stream,datagram} #291

Merged
merged 1 commit into from
Aug 26, 2022

Conversation

talex5
Copy link
Collaborator

@talex5 talex5 commented Aug 26, 2022

These filter the results of getaddrinfo to ensure you get something of the correct type. This makes it possible to use the result directly with e.g. connect.

This is also used in the tests to avoid non-deterministic ordering of TCP vs UDP results, which fixes #285.

/cc @bikallem

These filter the results of `getaddrinfo` to ensure you get something of
the correct type.

This is used in the tests to avoid non-deterministic ordering of TCP vs
UDP results.
@bikallem
Copy link
Contributor

Looks good. 👍

@talex5 talex5 merged commit b3955da into ocaml-multicore:main Aug 26, 2022
@talex5 talex5 deleted the getaddr-typed branch August 26, 2022 14:25
talex5 added a commit to talex5/opam-repository that referenced this pull request Aug 26, 2022
CHANGES:

New features:

- Add `Eio.Condition` (@TheLortex @talex5 ocaml-multicore/eio#277).
  Allows a fiber to wait for some condition to become true.

- Add `Eio.Net.getaddrinfo` and `getnameinfo` (@bikallem @talex5 ocaml-multicore/eio#278 ocaml-multicore/eio#288 ocaml-multicore/eio#291).
  Convert between host names and addresses.

- Add `Eio.Debug` (@talex5 ocaml-multicore/eio#276).
  Currently, this allows overriding the `traceln` function.

- `Buf_write.create`: make switch optional (@talex5 ocaml-multicore/eio#283).
  This makes things easier for people porting code from Faraday.

Bug fixes:

- Allow sharing of libuv poll handles (@patricoferris @talex5 ocaml-multicore/eio#279).
  Luv doesn't allow two callers to watch the same file handle, so we need to handle that in Eio.

Other changes:

- Upgrade to uring 0.4 (@talex5 ocaml-multicore/eio#290).

- Mention `Mutex`, `Semaphore` and `Condition` in the README (@talex5 ocaml-multicore/eio#281).
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.

None yet

2 participants