use ToolchainSelect instead of usize to select toolchain in ReportConfig

Author: zeegomo

src/report/analyzer.rs

@@ -7,9 +7,14 @@ use crate::results::{
 use std::collections::BTreeSet;
 use std::collections::HashMap;
 
+pub enum ToolchainSelect {
+    Start,
+    End,
+}
+
 pub enum ReportConfig {
     Simple,
-    Complete { toolchain: usize },
+    Complete(ToolchainSelect),
 }
 
 #[cfg_attr(test, derive(Debug, PartialEq))]
@@ -85,8 +90,10 @@ pub fn analyze_report(test: RawTestResults) -> TestResults {
 
     let mut categories = HashMap::new();
     for (cat, crates) in comparison {
-        if let ReportConfig::Complete { toolchain } = cat.report_config() {
-            categories.insert(cat, analyze_detailed(toolchain, crates));
+        if let ReportConfig::Complete(toolchain) = cat.report_config() {
+            // variants in an enum are numbered following an
+            // increasing sequence starting from 0
+            categories.insert(cat, analyze_detailed(toolchain as usize, crates));
         } else {
             categories.insert(cat, ReportCrates::Plain(crates));
         }

src/report/mod.rs

@@ -2,7 +2,7 @@ use crate::config::Config;
 use crate::crates::Crate;
 use crate::experiments::Experiment;
 use crate::prelude::*;
-use crate::report::analyzer::{analyze_report, ReportConfig};
+use crate::report::analyzer::{analyze_report, ReportConfig, ToolchainSelect};
 use crate::results::{EncodedLog, EncodingType, FailureReason, ReadResults, TestResult};
 use crate::toolchain::Toolchain;
 use crate::utils;
@@ -91,8 +91,8 @@ impl Comparison {
 
     pub fn report_config(self) -> ReportConfig {
         match self {
-            Comparison::Regressed => ReportConfig::Complete { toolchain: 1 },
-            Comparison::Fixed => ReportConfig::Complete { toolchain: 0 },
+            Comparison::Regressed => ReportConfig::Complete(ToolchainSelect::End),
+            Comparison::Fixed => ReportConfig::Complete(ToolchainSelect::Start),
             Comparison::Unknown
             | Comparison::Error
             | Comparison::SpuriousRegressed