implement IDeref, IBlockingDeref for channels
Description
Environment
Attachments
Activity
Alex Miller September 26, 2024 at 8:52 PM
This was an idea from Rich, discussed in Inside Transducers with a goal of “better integration with existing code”.
Stuart Halloway September 26, 2024 at 1:01 PM
What said, plus there is no problem statement here.
Kevin Downey September 26, 2024 at 12:36 AM
the semantics of this seem very questionable to me.
if deref == take! on a channel, then deref is mutating the channel, I don’t think any other uses of deref in clojure.core do that.
Alex Miller January 28, 2015 at 8:23 PM
That doesn't seem to work to me. If you have some formulation that would, that would be great though!
Fogus January 9, 2015 at 7:11 PM
> However, I'm a little stumped on how to do the equivalent with
> alt!
!.
I wonder if the following will solve the alts!! problem:
And then using it in the deref impl behind IBlockingDeref. As it stands it's difficult to grok the code as written given that it's using some core.async implementation details (indeed, the same code). Whereas your implementation of IBlockingDeref.deref is crystal clear, the implementation prompted by the cyclic dependency is... less so. Not to belabor the point (which I seem to be), but alts!! is a clear implementation for this, but using the guts muddies the water. If I had my druthers, I'd prefer clarity.
That said, being a human macro-expander I was able to reason through the implementation eventually.
Details
Assignee
UnassignedUnassignedReporter
Stuart HallowayStuart HallowayApproval
VettedPatch
Code and TestPriority
Major
Details
Details
Assignee
Reporter
Stuart HallowayApproval
Patch
Priority
Major
each deref will take a value from the channel
deref from a closed channel returns nil
deref with timeout must either timeout or take value from the channel (not both!). Might just want to call some variant of alt!! through the var
use internals efficiently
do NOT implement IPending/realized, neither interpretation is that great
Patch: async-102-2.patch
The implementation of deref and timed deref is conceptually straightforward, but is difficult to implement due to circular namespace dependencies. Suggestions welcome.