From 0e4f7a3846527673977eedcbfc2b2478352d4de7 Mon Sep 17 00:00:00 2001 From: Christopher Berner Date: Sat, 14 Aug 2021 16:23:56 -0700 Subject: [PATCH] Switch some assert!s to debug_assert! Improves lookup performance by a couple percent --- src/btree.rs | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/btree.rs b/src/btree.rs index 5cd02df7..7b3db9ca 100644 --- a/src/btree.rs +++ b/src/btree.rs @@ -555,7 +555,7 @@ struct InternalAccessor<'a: 'b, 'b> { impl<'a: 'b, 'b> InternalAccessor<'a, 'b> { fn new(page: &'b PageImpl<'a>) -> Self { - assert_eq!(page.memory()[0], INTERNAL); + debug_assert_eq!(page.memory()[0], INTERNAL); InternalAccessor { page } } @@ -580,7 +580,7 @@ impl<'a: 'b, 'b> InternalAccessor<'a, 'b> { max_child = mid; } } - assert_eq!(min_child, max_child); + debug_assert_eq!(min_child, max_child); self.child_page(min_child).unwrap() } @@ -596,7 +596,7 @@ impl<'a: 'b, 'b> InternalAccessor<'a, 'b> { } fn table_id(&self, n: usize) -> Option { - assert!(n < BTREE_ORDER - 1); + debug_assert!(n < BTREE_ORDER - 1); let len = self.key_len(n); if len == 0 { return None; @@ -608,7 +608,7 @@ impl<'a: 'b, 'b> InternalAccessor<'a, 'b> { } fn table_and_key(&self, n: usize) -> Option<(u64, &[u8])> { - assert!(n < BTREE_ORDER - 1); + debug_assert!(n < BTREE_ORDER - 1); let len = self.key_len(n); if len == 0 { return None; @@ -621,7 +621,7 @@ impl<'a: 'b, 'b> InternalAccessor<'a, 'b> { } fn key(&self, n: usize) -> Option<&[u8]> { - assert!(n < BTREE_ORDER - 1); + debug_assert!(n < BTREE_ORDER - 1); let offset = self.key_offset(n); let len = self.key_len(n); if len == 0 { @@ -631,7 +631,7 @@ impl<'a: 'b, 'b> InternalAccessor<'a, 'b> { } fn child_page(&self, n: usize) -> Option { - assert!(n < BTREE_ORDER); + debug_assert!(n < BTREE_ORDER); if n > 0 && self.key_len(n - 1) == 0 { return None; }