Fix linting issues

cloudflare · Nov 28, 2024 · e22d84d · e22d84d
1 parent f67bcf0
commit e22d84d
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 9 deletions.
diff --git a/crates/daphne/src/lib.rs b/crates/daphne/src/lib.rs
@@ -322,10 +322,12 @@ impl std::fmt::Display for DapBatchBucket {
     }
 }
 
+type TimestampedReportList = Vec<(ReportId, Time)>;
+
 /// A set of values related to reports in the same bucket.
 #[derive(Debug)]
 pub struct DapAggregateSpan<T> {
-    span: HashMap<DapBatchBucket, (T, Vec<(ReportId, Time)>)>,
+    span: HashMap<DapBatchBucket, (T, TimestampedReportList)>,
 }
 
 // We can't derive default because it will require T to be Default, which we don't need.
@@ -338,7 +340,7 @@ impl<T> Default for DapAggregateSpan<T> {
 }
 
 impl<T> IntoIterator for DapAggregateSpan<T> {
-    type IntoIter = <HashMap<DapBatchBucket, (T, Vec<(ReportId, Time)>)> as IntoIterator>::IntoIter;
+    type IntoIter = <HashMap<DapBatchBucket, (T, TimestampedReportList)> as IntoIterator>::IntoIter;
 
     type Item = <Self::IntoIter as Iterator>::Item;
 
@@ -440,26 +442,26 @@ impl<T> DapAggregateSpan<T> {
     }
 
     /// Return an iterator over the aggregate span.
-    pub fn iter(&self) -> impl Iterator<Item = (&DapBatchBucket, &(T, Vec<(ReportId, Time)>))> {
+    pub fn iter(&self) -> impl Iterator<Item = (&DapBatchBucket, &(T, TimestampedReportList))> {
         self.span.iter()
     }
 }
 
-impl<T> FromIterator<(DapBatchBucket, (T, Vec<(ReportId, Time)>))> for DapAggregateSpan<T> {
+impl<T> FromIterator<(DapBatchBucket, (T, TimestampedReportList))> for DapAggregateSpan<T> {
     fn from_iter<I>(iter: I) -> Self
     where
-        I: IntoIterator<Item = (DapBatchBucket, (T, Vec<(ReportId, Time)>))>,
+        I: IntoIterator<Item = (DapBatchBucket, (T, TimestampedReportList))>,
     {
         Self {
             span: iter.into_iter().collect(),
         }
     }
 }
 
-impl<T> Extend<(DapBatchBucket, (T, Vec<(ReportId, Time)>))> for DapAggregateSpan<T> {
+impl<T> Extend<(DapBatchBucket, (T, TimestampedReportList))> for DapAggregateSpan<T> {
     fn extend<I>(&mut self, iter: I)
     where
-        I: IntoIterator<Item = (DapBatchBucket, (T, Vec<(ReportId, Time)>))>,
+        I: IntoIterator<Item = (DapBatchBucket, (T, TimestampedReportList))>,
     {
         self.span.extend(iter);
     }

diff --git a/crates/daphne/src/messages/mod.rs b/crates/daphne/src/messages/mod.rs
@@ -142,7 +142,7 @@ mod base64url_bytes {
         O: From<[u8; N]>,
     {
         struct Visitor<const N: usize, O>(std::marker::PhantomData<[O; N]>);
-        impl<'de, const N: usize, O> de::Visitor<'de> for Visitor<N, O>
+        impl<const N: usize, O> de::Visitor<'_> for Visitor<N, O>
         where
             O: From<[u8; N]>,
         {

diff --git a/crates/daphne/src/pine/mod.rs b/crates/daphne/src/pine/mod.rs
@@ -359,7 +359,7 @@ fn bits(x: u64) -> usize {
 }
 
 fn chunk_count(chunk_length: usize, length: usize) -> usize {
-    (length + chunk_length - 1) / chunk_length
+    length.div_ceil(chunk_length)
 }
 
 fn dst(algorithm_id: u32, usage: u16) -> [u8; DST_SIZE] {