-
Notifications
You must be signed in to change notification settings - Fork 2
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
remove unused ExecutionResult::create_address #169
Conversation
A few questions:
athena/ffi/athcon/include/athcon/athcon.h Lines 394 to 403 in 46a7084
This is a nuance involving gas pricing and the cost of state access. In a single transaction, we need to consider the fact that the first time an account is touched (or created), the cost is higher than subsequent accesses (this is what is meant by "warm"). We probably don't need to worry about this for now, but I just want to make sure that we're aware of it. |
I can't answer any of these questions as we don't support these functionalities at the moment nor are they designed. However, there is no point in having a hypothetically useful thing "for the future". There's a high chance that, when the time to implement these features comes, we will need to find a different, tailored for our needs, solution anyway. |
Let me give it more thought. I think we could return the spawned addresses in every case "normally" by bubbling it up the call stack, if the
This program could return
The address(es) could be simply returned from the
I don't understand the use case described in the comment. What address would it return if spawning failed? Anyhow, probably the same method could be used (return the address in execution results). |
Only one thing to add: the result of each |
The address of the spawned program is returned the same way as the results of all contract methods via the
ExecutionResult::output
.