Skip to content

Commit c7686fe

Browse files
committed
style(nodedb-fts): use size_of shorthand in compaction memory reserves
Replace std::mem::size_of::<T>() with the size_of::<T>() edition 2021 import shorthand across the two governor.reserve calls in compact_segments.
1 parent b16b215 commit c7686fe

1 file changed

Lines changed: 4 additions & 10 deletions

File tree

‎nodedb-fts/src/lsm/compaction.rs‎

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -113,22 +113,16 @@ pub fn compact_level<B: FtsBackend>(
113113

114114
let _readers_guard = governor
115115
.map(|gov| {
116-
gov.reserve(
117-
EngineId::Fts,
118-
to_merge.len() * std::mem::size_of::<SegmentReader>(),
119-
)
120-
.map_err(CompactError::Budget)
116+
gov.reserve(EngineId::Fts, to_merge.len() * size_of::<SegmentReader>())
117+
.map_err(CompactError::Budget)
121118
})
122119
.transpose()?;
123120
let mut readers = Vec::with_capacity(to_merge.len());
124121

125122
let _ids_guard = governor
126123
.map(|gov| {
127-
gov.reserve(
128-
EngineId::Fts,
129-
to_merge.len() * std::mem::size_of::<String>(),
130-
)
131-
.map_err(CompactError::Budget)
124+
gov.reserve(EngineId::Fts, to_merge.len() * size_of::<String>())
125+
.map_err(CompactError::Budget)
132126
})
133127
.transpose()?;
134128
let mut merged_ids = Vec::with_capacity(to_merge.len());

0 commit comments

Comments
 (0)