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

Important missing case #131

Closed
erights opened this issue Jul 11, 2019 · 1 comment
Closed

Important missing case #131

erights opened this issue Jul 11, 2019 · 1 comment
Labels
eventual-send package: eventual-send

Comments

@erights
Copy link
Member

erights commented Jul 11, 2019

https://github.com/Agoric/eventual-send/blob/443da01a45937b77b95002b9d8d0372e8269839c/src/index.js#L163

If resolvedHandler was absent and presence is a handled promise (which is valid. See https://github.com/Agoric/eventual-send/issues/13 ), then this promise should share that promise's handler. If it is a resolved handled promise, then it should share that promise's presence, but the presence's canonical remote promise should not be modified.

@warner
Copy link
Member

warner commented Dec 4, 2019

in the old repo. this was eventual-send issue 14

@warner warner transferred this issue from Agoric/eventual-send Dec 4, 2019
@warner warner added the eventual-send package: eventual-send label Dec 4, 2019
dckc pushed a commit to dckc/agoric-sdk that referenced this issue Dec 5, 2019
This, and default-evaluate-options-0.1.3, should fix Agoric#131, by making
liveSlots tell the kernel about pipelined messages. We won't get actual
pipelining until we add Agoric#88, to enable the kernel to deliver pipelined
messages to the comms vat.
dckc pushed a commit to dckc/agoric-sdk that referenced this issue Dec 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
eventual-send package: eventual-send
Projects
None yet
Development

No branches or pull requests

2 participants