We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
stream::Shared
Currently, we hold a stream::Shared within the stream and one in Active. This is shared by being wrapped in Arc<Mutex>.
Active
Arc<Mutex>
It would be slightly more ergonomic for both usage sites if the Arc<Mutex> would be inside stream::Shared.
The text was updated successfully, but these errors were encountered:
Hey @mxinden
Is this issue still available for pickup?
Sorry, something went wrong.
Yes it is @akaladarshi :)
@dariusc93 Can you assign it to me?.
Also just to get the full idea: We need to put Arc<Mutex> inside stream::Shared, so we don't have to use Arc<Mutex<Shared>> everywhere else.
Arc<Mutex<Shared>>
Successfully merging a pull request may close this issue.
Currently, we hold a
stream::Shared
within the stream and one inActive
. This is shared by being wrapped inArc<Mutex>
.It would be slightly more ergonomic for both usage sites if the
Arc<Mutex>
would be insidestream::Shared
.The text was updated successfully, but these errors were encountered: