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

Implement Sink trait based upon work in tide-websockets PRs #3 and #9. #17

Closed
wants to merge 1 commit into from

Conversation

cryptoquick
Copy link
Contributor

This work should be included to support shared mutable references across thread contexts.

An example is included from before, but for a more practical example, please see:

ChainSafe/forest@ff2691b#diff-2770a30d9f259666fb470d6f11cf1851ebb2d579a1480a8173d3855572748385

And in particular, this case:

https://github.com/ChainSafe/forest/blob/ff2691bab92823a8595d1d456ed5fa9683641d76/node/rpc/src/rpc_ws_handler.rs#L33

See also: #3 #9

@jbr
Copy link
Member

jbr commented Mar 11, 2021

See #9 (comment)

@jbr jbr closed this Mar 11, 2021
@cryptoquick
Copy link
Contributor Author

@jbr I don't understand how that solves my use-case. Can you explain how I would adjust my code to use your library?

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.

2 participants