chore: helper function effective_balance_spending_without_value#3030
chore: helper function effective_balance_spending_without_value#3030
Conversation
CodSpeed Performance ReportMerging #3030 will not alter performanceComparing Summary
|
mattsse
left a comment
There was a problem hiding this comment.
makes sense to me, some doc nits
klkvr
left a comment
There was a problem hiding this comment.
I'm personally fine with keeping this duplicated
|
|
I meant that this is just a single line of code deduplicating which might not be worth the indirection and a new trait method |
|
I see, as it is a trait method with a default implementation, it will not require reimplementation it. For this case it made sense to add it. It does replace fetching two values from the same place |
* chore: helper function effective_balance_spending_without_value * add box * rm function that is part of different PR * nits and cleanup * gas_balance_spending
Helper function as we have three places where value needs to be deducted.
Exctracted from #2991