Fix int_plus_one warnings

Signed-off-by: Jacinta Ferrant <[email protected]>

Author: jferrant

stackslib/src/chainstate/nakamoto/shadow.rs

@@ -632,8 +632,8 @@ impl NakamotoBlockBuilder {
             ))
         })?;
 
-        if naka_tip_header.anchored_header.height() + 1
-            <= naka_tip_tenure_start_header.anchored_header.height()
+        if naka_tip_header.anchored_header.height()
+            < naka_tip_tenure_start_header.anchored_header.height()
         {
             return Err(Error::InvalidStacksBlock(
                 "Nakamoto tip is lower than its tenure-start block".into(),

stackslib/src/chainstate/stacks/index/trie.rs

@@ -378,7 +378,7 @@ impl Trie {
         // append the new leaf and the end of the file.
         let new_leaf_disk_ptr = storage.last_ptr()?;
         let new_leaf_chr = cursor.path[cursor.tell()]; // NOTE: this is safe because !cursor.eop()
-        let new_leaf_path = cursor.path[(if cursor.tell() + 1 <= cursor.path.len() {
+        let new_leaf_path = cursor.path[(if cursor.tell() < cursor.path.len() {
             cursor.tell() + 1
         } else {
             cursor.path.len()

stackslib/src/chainstate/stacks/tests/block_construction.rs

@@ -2327,7 +2327,7 @@ fn test_build_anchored_blocks_invalid() {
 
                 eprintln!("\n\nat resume parent tenure:\nlast_parent: {:?}\nlast_parent_tip: {:?}\n\n", &last_parent, &last_parent_tip);
             }
-            else if tenure_id >= bad_block_tenure + 1 {
+            else if tenure_id > bad_block_tenure {
                 last_parent = None;
                 last_parent_tip = None;
             }

stackslib/src/net/relay.rs

@@ -286,7 +286,7 @@ impl RelayerStats {
                     let mut to_remove = vec![];
                     for (ts, old_addr) in self.relay_updates.iter() {
                         self.relay_stats.remove(old_addr);
-                        if self.relay_stats.len() <= MAX_RELAYER_STATS - 1 {
+                        if self.relay_stats.len() < MAX_RELAYER_STATS {
                             break;
                         }
                         to_remove.push(*ts);
@@ -342,7 +342,7 @@ impl RelayerStats {
                 let mut to_remove = vec![];
                 for (ts, old_nk) in self.recent_updates.iter() {
                     self.recent_messages.remove(old_nk);
-                    if self.recent_messages.len() <= (MAX_RELAYER_STATS as usize) - 1 {
+                    if self.recent_messages.len() < MAX_RELAYER_STATS {
                         break;
                     }
                     to_remove.push(*ts);