fix: off by one error in average_case_delete_operations_for_delete_up…

…_tree_while_empty
dashpay · Dec 13, 2022 · 6864631 · 6864631
1 parent 5d2c15e
commit 6864631
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/grovedb/src/operations/delete.rs b/grovedb/src/operations/delete.rs
@@ -671,7 +671,7 @@ impl GroveDb {
                     is_sum_tree,
                 ) = cost_return_on_error_no_add!(
                     &cost,
-                    if height == path_len {
+                    if height == path_len - 1 {
                         if let Some(layer_info) = estimated_layer_info.get(height as u64) {
                             let estimated_value_len = cost_return_on_error_no_add!(
                                 &cost,
@@ -715,7 +715,7 @@ impl GroveDb {
                                 layer_info.is_sum_tree,
                             ))
                         } else {
-                            Err(Error::InvalidParameter("intermediate flag size missing"))
+                            Err(Error::InvalidParameter("intermediate layer info missing"))
                         }
                     }
                 );

diff --git a/merk/src/estimated_costs/average_case_costs.rs b/merk/src/estimated_costs/average_case_costs.rs
@@ -112,7 +112,7 @@ impl EstimatedLayerSizes {
             },
             _ => {
                 return Err(Error::WrongEstimatedCostsElementTypeForLevel(
-                    "this layer is needs to have trees",
+                    "this layer needs to have trees",
                 ));
             }
         }