Skip to content

Refactor the sockets_utils.h headers#729

Open
alexcrichton wants to merge 1 commit intoWebAssembly:mainfrom
alexcrichton:refactor-socket-utis
Open

Refactor the sockets_utils.h headers#729
alexcrichton wants to merge 1 commit intoWebAssembly:mainfrom
alexcrichton:refactor-socket-utis

Conversation

@alexcrichton
Copy link
Collaborator

This commit refactors the helpers for TCP/UDP which predated much of the work around WASIp2-natively-in-libc. Changes here are:

  • Helpers to use __wasilibc_* prefix that many other internal functions in wasi-libc use.
  • Helpers directly set errno instead of returning a bool and requiring the caller to set errno.
  • Some helpers were renamed/tweaked to be a bit more idiomatic with C.

This commit refactors the helpers for TCP/UDP which predated much of the
work around WASIp2-natively-in-libc. Changes here are:

* Helpers to use `__wasilibc_*` prefix that many other internal
  functions in wasi-libc use.
* Helpers directly set `errno` instead of returning a `bool` and
  requiring the caller to set `errno`.
* Some helpers were renamed/tweaked to be a bit more idiomatic with C.
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.

1 participant