Use FxHash for faster entry hashing

projectfluent · Jan 22, 2021 · b16ab3a · b16ab3a
1 parent 6d5855e
commit b16ab3a
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/fluent-bundle/Cargo.toml b/fluent-bundle/Cargo.toml
@@ -33,6 +33,7 @@ ouroboros = "0.7"
 smallvec = "1"
 unic-langid = "0.9"
 intl-memoizer = { version = "0.5", path = "../intl-memoizer" }
+rustc-hash = "1"
 
 [dev-dependencies]
 criterion = "0.3"

diff --git a/fluent-bundle/src/bundle.rs b/fluent-bundle/src/bundle.rs
@@ -6,7 +6,8 @@
 
 use std::borrow::Borrow;
 use std::borrow::Cow;
-use std::collections::hash_map::{Entry as HashEntry, HashMap};
+use rustc_hash::FxHashMap;
+use std::collections::hash_map::Entry as HashEntry;
 use std::default::Default;
 use std::fmt;
 
@@ -31,7 +32,7 @@ use crate::types::FluentValue;
 pub struct FluentBundleBase<R, M> {
     pub locales: Vec<LanguageIdentifier>,
     pub(crate) resources: Vec<R>,
-    pub(crate) entries: HashMap<String, Entry>,
+    pub(crate) entries: FxHashMap<String, Entry>,
     pub(crate) intls: M,
     pub(crate) use_isolating: bool,
     pub(crate) transform: Option<fn(&str) -> Cow<str>>,
@@ -64,7 +65,7 @@ impl<R, M: MemoizerKind> FluentBundleBase<R, M> {
         Self {
             locales,
             resources: vec![],
-            entries: HashMap::new(),
+            entries: FxHashMap::default(),
             intls: M::new(first_locale),
             use_isolating: true,
             transform: None,