xtra icon indicating copy to clipboard operation
xtra copied to clipboard

Published 20 hours ago • verified publisher icon Restioson

PoC: Introduce `Mailbox` and make `Context` short-lived

Open thomaseizinger opened this issue 3 years ago • 1 comments

This is a PoC for #126. It didn't quite turn out the way I wanted but I am putting it up anyway so gather some feedback.

thomaseizinger avatar Jul 16 '22 17:07 thomaseizinger

One thing that would be cool to do on top of this is to remove the Sender and Receiver types so we have one less layer between the internal channel implementation and the public types.

thomaseizinger avatar Jul 16 '22 18:07 thomaseizinger

This is currently blocked by these PRs:

  • https://github.com/Restioson/xtra/pull/158
  • https://github.com/Restioson/xtra/pull/159

thomaseizinger avatar Aug 31 '22 18:08 thomaseizinger

This is currently blocked by these PRs:

* [Introduce generic `ChanPtr` #158](https://github.com/Restioson/xtra/pull/158)

* [Introduce top-level `recv_future` module #159](https://github.com/Restioson/xtra/pull/159)

I've squash-merged both PRs into here so that I could resolve the merge conflicts already.

thomaseizinger avatar Aug 31 '22 19:08 thomaseizinger

Is this PR ready?

Restioson avatar Sep 08 '22 20:09 Restioson

Is this PR ready?

Yes!

thomaseizinger avatar Sep 08 '22 23:09 thomaseizinger

Is this PR ready?

It probably needs a changelog entry before we can merge it though. Plus the README needs to be adjusted too!

thomaseizinger avatar Sep 09 '22 07:09 thomaseizinger