Comment added

paritytech · Jul 10, 2024 · 45603b8 · 45603b8
1 parent 3a8c2d2
commit 45603b8
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/substrate/frame/contracts/src/wasm/runtime.rs b/substrate/frame/contracts/src/wasm/runtime.rs
@@ -254,6 +254,10 @@ pub enum RuntimeCosts {
 	UnlockDelegateDependency,
 }
 
+// For the function that modifies the storage, the benchmarks are done with one item in the
+// transient_storage (BTreeMap). To consider the worst-case scenario, the weight of the overhead of
+// writing to a full BTreeMap should be included. On top of that, the rollback weight is added,
+// which is the worst scenario.
 macro_rules! cost_write {
 	// cost_write!(name, a, b, c) -> T::WeightInfo::name(a, b, c).saturating_add(T::WeightInfo::rollback_transient_storage())
 	// .saturating_add(T::WeightInfo::set_transient_storage_full().saturating_sub(T::WeightInfo::set_transient_storage_empty())