Skip to content

Commit 99e1514

Browse files
committed
clippy
Signed-off-by: Jesse Szwedko <[email protected]>
1 parent 5297221 commit 99e1514

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

src/transforms/tag_cardinality_limit/mod.rs

+3-3
Original file line numberDiff line numberDiff line change
@@ -55,11 +55,11 @@ impl TagCardinalityLimit {
5555
}
5656

5757
// Tag value not yet part of the accepted set.
58-
if tag_value_set.len() < self.config.value_limit as usize {
58+
if tag_value_set.len() < self.config.value_limit {
5959
// accept the new value
6060
tag_value_set.insert(value.clone());
6161

62-
if tag_value_set.len() == self.config.value_limit as usize {
62+
if tag_value_set.len() == self.config.value_limit {
6363
emit!(TagCardinalityValueLimitReached { key });
6464
}
6565

@@ -76,7 +76,7 @@ impl TagCardinalityLimit {
7676
self.accepted_tags
7777
.get(key)
7878
.map(|value_set| {
79-
!value_set.contains(value) && value_set.len() >= self.config.value_limit as usize
79+
!value_set.contains(value) && value_set.len() >= self.config.value_limit
8080
})
8181
.unwrap_or(false)
8282
}

src/transforms/tag_cardinality_limit/tag_value_set.rs

+2-2
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ impl fmt::Debug for TagValueSetStorage {
2727
impl AcceptedTagValueSet {
2828
pub fn new(value_limit: usize, mode: &Mode) -> Self {
2929
let storage = match &mode {
30-
Mode::Exact => TagValueSetStorage::Set(HashSet::with_capacity(value_limit as usize)),
30+
Mode::Exact => TagValueSetStorage::Set(HashSet::with_capacity(value_limit)),
3131
Mode::Probabilistic(config) => {
3232
let num_bits = config.cache_size_per_key / 8; // Convert bytes to bits
3333
TagValueSetStorage::Bloom(BloomFilter::with_size(num_bits))
@@ -39,7 +39,7 @@ impl AcceptedTagValueSet {
3939
pub fn contains(&self, value: &TagValueSet) -> bool {
4040
match &self.storage {
4141
TagValueSetStorage::Set(set) => set.contains(value),
42-
TagValueSetStorage::Bloom(bloom) => bloom.contains(&value),
42+
TagValueSetStorage::Bloom(bloom) => bloom.contains(value),
4343
}
4444
}
4545

0 commit comments

Comments
 (0)