Merge pull request #1371 from jqnatividad/apply-perfect-hash-function…

…s-operations `apply` & `applydp`: use perfect hash functions for operations lookups
dathere · Oct 18, 2023 · 539ef79 · 539ef79
2 parents 6912b18 + 01a459a
commit 539ef79
Show file tree

Hide file tree

Showing 4 changed files with 58 additions and 11 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -193,7 +193,7 @@ simple-home-dir = { version = "0.2", features = [
 smartstring = { version = "1", optional = true }
 snap = "1"
 strsim = { version = "0.10", optional = true }
-strum = "0.25"
+strum = { version = "0.25", features = ["phf"] }
 strum_macros = "0.25"
 sysinfo = "0.29"
 tabwriter = "1.3"

diff --git a/src/cmd/apply.rs b/src/cmd/apply.rs
@@ -377,7 +377,8 @@ use crate::{
     util, CliResult,
 };
 
-#[derive(EnumString)]
+#[derive(Clone, EnumString)]
+#[strum(use_phf)]
 #[strum(ascii_case_insensitive)]
 #[allow(non_camel_case_types)]
 enum Operations {

diff --git a/src/cmd/applydp.rs b/src/cmd/applydp.rs
@@ -253,7 +253,8 @@ use crate::{
     util, CliResult,
 };
 
-#[derive(EnumString)]
+#[derive(Clone, EnumString)]
+#[strum(use_phf)]
 #[strum(ascii_case_insensitive)]
 #[allow(non_camel_case_types)]
 enum Operations {