no more trailing endlines

private-attribution · Sep 19, 2024 · 7fd93e1 · 7fd93e1
1 parent 95384a1
commit 7fd93e1
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 10 deletions.
diff --git a/ipa-core/src/bin/report_collector.rs b/ipa-core/src/bin/report_collector.rs
@@ -25,7 +25,6 @@ use ipa_core::{
     net::MpcHelperClient,
     report::{EncryptedOprfReportStreams, DEFAULT_KEY_ID},
     test_fixture::{
-        hybrid::{hybrid_in_the_clear, TestHybridRecord},
         ipa::{ipa_in_the_clear, CappingOrder, IpaSecurityModel, TestRawDataRecord},
         EventGenerator, EventGeneratorConfig, HybridEventGenerator, HybridGeneratorConfig,
     },
@@ -242,9 +241,7 @@ fn gen_hybrid_inputs(
     let rng = seed
         .map(StdRng::seed_from_u64)
         .unwrap_or_else(StdRng::from_entropy);
-    let event_gen = HybridEventGenerator::with_config(rng, args)
-        .take(count as usize)
-        .collect::<Vec<_>>();
+    let event_gen = HybridEventGenerator::with_config(rng, args).take(count as usize);
 
     let mut writer: Box<dyn Write> = if let Some(path) = output_file {
         Box::new(OpenOptions::new().write(true).create_new(true).open(path)?)

diff --git a/ipa-core/src/cli/csv.rs b/ipa-core/src/cli/csv.rs
@@ -29,14 +29,10 @@ impl Serializer for crate::test_fixture::hybrid::TestHybridRecord {
                 match_key,
                 breakdown_key,
             } => {
-                write!(buf, "i,")?;
-                write!(buf, "{match_key},")?;
-                write!(buf, "{breakdown_key},")?;
+                write!(buf, "i,{match_key},{breakdown_key}")?;
             }
             crate::test_fixture::hybrid::TestHybridRecord::TestConversion { match_key, value } => {
-                write!(buf, "c,")?;
-                write!(buf, "{match_key},")?;
-                write!(buf, "{value}")?;
+                write!(buf, "c,{match_key},{value}")?;
             }
         }