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.

Sign up for GitHub

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

Jump to bottom

Acceptance as an IPC call #1007

Closed
jjcnn opened this issue Jun 10, 2021 · 1 comment
Closed

Acceptance as an IPC call #1007

jjcnn opened this issue Jun 10, 2021 · 1 comment
Labels
blockchain infrastructure

Comments

@jjcnn
Copy link
Contributor

jjcnn commented Jun 10, 2021

The blockchain handles transition execution as a black box, so when a contract performs an accept statement the blockchain doesn't immediately transfer the funds from _sender to _this_address. Rather, the funds are not transferred until the transition completes successfully.

This means that Scilla needs to "mask" the remote read of _sender._balance, because the underlying IPC fetch gives us the balance without the deduction of the accepted funds.

It would be much easier if we could invoke the transfer when Scilla executed an accept statement, so that we could simply read the balance directly without having to jump through any hoops.
vaivaswatha added a commit to Zilliqa/scilla-rtl that referenced this issue Jun 24, 2021
@vaivaswatha
IPC fetch needs to special case _balance
c30106c 
Fixes #34. Based on how Zilliqa/scilla#1007
goes, we may want to remove this special handling and
communicate `accept` directly to the blockchain.
@jjcnn
Copy link
Contributor Author

jjcnn commented Jul 14, 2021

Made obsolete by #1032

@jjcnn jjcnn closed this as completed Jul 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
blockchain infrastructure
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jjcnn