[pbs-devel] [PATCH v2 proxmox-backup 2/4] tools: lru cache: document limitations for cache capacity
Christian Ebner
c.ebner at proxmox.com
Mon May 19 07:55:16 CEST 2025
Since commit 1e7639bf ("fixup minimum lru capacity") the minimum
cache capacity is forced to be 1 to bypass edge cases for it being 0.
Explicitly mention this in the doc comment, as this behavior can be
unexpected.
Signed-off-by: Christian Ebner <c.ebner at proxmox.com>
---
changes since version 1:
- not present in previous version
pbs-tools/src/lru_cache.rs | 2 ++
1 file changed, 2 insertions(+)
diff --git a/pbs-tools/src/lru_cache.rs b/pbs-tools/src/lru_cache.rs
index 9e0112647..94757bbf7 100644
--- a/pbs-tools/src/lru_cache.rs
+++ b/pbs-tools/src/lru_cache.rs
@@ -121,6 +121,8 @@ impl<K, V> LruCache<K, V> {
impl<K: std::cmp::Eq + std::hash::Hash + Copy, V> LruCache<K, V> {
/// Create LRU cache instance which holds up to `capacity` nodes at once.
+ ///
+ /// Forces a minimum `capacity` of 1 in case of the given value being 0.
pub fn new(capacity: usize) -> Self {
let capacity = capacity.max(1);
Self {
--
2.39.5
More information about the pbs-devel
mailing list