Skip to content
This repository has been archived by the owner on Jul 31, 2018. It is now read-only.

uv_link_t: initial #32

Closed
wants to merge 7 commits into from
Closed

uv_link_t: initial #32

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
167660b
uv_link_t: initial
indutny Jun 3, 2016
561ba80
fixes
indutny Jun 3, 2016
50199ab
add note about observability
indutny Jun 3, 2016
d4ac1d5
fixes
indutny Jun 3, 2016
2dd1e26
fix typo
indutny Jun 4, 2016
8080c8f
fix
indutny Jun 6, 2016
6c683d0
add description
indutny Jun 16, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 2 additions & 3 deletions xxx-uv_link_t.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -38,9 +38,8 @@ From [`uv_link_t`][0] readme:
high-performant way using very narrow interfaces.

Given that [`uv_link_t`][0] depends only on [libuv][1], it is very easy to
envision how it will be used in Node.js addons. Even core modules will no longer
be required to be distributed with core, since they could be built without core
dependencies.
envision how it will be used in Node.js addons. Even core modules will could
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"will could" -> "could". Other than that minor thing, reads great. Thanks!

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ack.

be reimplemented on a user level without a need to patch the core itself.

Additionally, with the `uv_link_observer_t` (which is a part of [`uv_link_t`][0]
distribution), all of the reads that right now happen internally within the
Expand Down