-
Notifications
You must be signed in to change notification settings - Fork 712
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
Deferred calls #4723
base: mainnet_2_3
Are you sure you want to change the base?
Deferred calls #4723
Conversation
move reset of addr stack after event_emit
remove reset of created_event_index move event_emit
.storage_costs_constants | ||
.ledger_cost_per_byte | ||
.saturating_mul_u64(call.parameters.len() as u64); | ||
context.transfer_coins(None, Some(call.sender_address), amount, false)?; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
in what case does the "?" trigger in this case ?
if call.cancelled { | ||
Ok(result) | ||
} else { | ||
let deferred_call_execution = || { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe extract this to another function for better clarity
@@ -486,10 +493,22 @@ impl SpeculativeDeferredCallRegistry { | |||
self.config.slot_overbooking_penalty, // total_initial_coin_supply/10000 | |||
)?; | |||
|
|||
// Storage cost for the parameters | |||
let storage_cost = self |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add total cost of call
/// deferred call min gas increment (1 nanomassa) | ||
pub const DEFERRED_CALL_MIN_GAS_INCREMENT: u64 = 1; | ||
/// deferred call max gas cost (10 nanomassa) | ||
pub const DEFERRED_CALL_MIN_GAS_COST: u64 = 10; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this DEFERRED_CALL_MIN_GAS_COST or the MAX as written in the commented part?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this DEFERRED_CALL_MIN_GAS_COST or the MAX as written in the commented part?
Hello @plouis01, indeed it is an error in the comment, I will update it
resync_check
flag