From b53aa2dd7d137fb7b031f3dc7dcde6241c8cbc9b Mon Sep 17 00:00:00 2001 From: Lars Wirzenius Date: Sun, 5 Dec 2021 18:49:23 +0200 Subject: chore: drop unnecessary code Sponsored-by: author --- src/specification.rs | 29 ++++++----------------------- 1 file changed, 6 insertions(+), 23 deletions(-) diff --git a/src/specification.rs b/src/specification.rs index 6bb72bc..c653735 100644 --- a/src/specification.rs +++ b/src/specification.rs @@ -98,8 +98,7 @@ impl Specification { fn check(&self) -> Result<(), SpecificationError> { let mut names = HashSet::new(); - for name in self.benchmarks.iter().map(|b| b.name()) { - let name = name.to_string(); + for name in self.benchmarks.iter().map(|b| b.benchmark.to_string()) { if names.contains(&name) { return Err(SpecificationError::DuplicateBenchmark(name)); } @@ -112,34 +111,18 @@ impl Specification { pub fn steps(&self) -> Vec { let mut steps = vec![]; for b in self.benchmarks.iter() { - steps.push(Step::Start(b.name().to_string())); - for (i, backup) in b.backups().enumerate() { - for change in backup.changes() { + steps.push(Step::Start(b.benchmark.to_string())); + for (i, backup) in b.backups.iter().enumerate() { + for change in backup.changes.iter() { steps.push(Step::from(change)); } steps.push(Step::Backup(i)); } - for (i, _) in b.backups().enumerate() { + for (i, _) in b.backups.iter().enumerate() { steps.push(Step::Restore(i)); } - steps.push(Step::Stop(b.name().to_string())); + steps.push(Step::Stop(b.benchmark.to_string())); } steps } } - -impl Benchmark { - pub fn name(&self) -> &str { - &self.benchmark - } - - pub fn backups(&self) -> impl Iterator { - self.backups.iter() - } -} - -impl Backup { - pub fn changes(&self) -> impl Iterator { - self.changes.iter() - } -} -- cgit v1.2.1