Move out bidi characters to constant

mullvad · Aug 30, 2024 · ad500fe · ad500fe
1 parent 53ca585
commit ad500fe
Showing 1 changed file with 7 additions and 6 deletions.
diff --git a/src/rules.rs b/src/rules.rs
@@ -1,3 +1,9 @@
+/// List of bidirectional formatting characters from https://en.wikipedia.org/wiki/Trojan_Source
+const BIDI_CHARACTERS: &[char] = &[
+    '\u{202A}', '\u{202b}', '\u{202c}', '\u{202d}', '\u{202e}', '\u{2066}', '\u{2067}', '\u{2068}',
+    '\u{2069}',
+];
+
 pub enum Decision {
     Allow,
     Deny,
@@ -59,12 +65,7 @@ impl CharacterType {
         match self {
             Self::CodePoint(rule_char) => *rule_char == c,
             Self::Range(range) => range.contains(&c),
-            Self::Bidi => [
-                // List of bidirectional formatting characters from https://en.wikipedia.org/wiki/Trojan_Source
-                '\u{202A}', '\u{202b}', '\u{202c}', '\u{202d}', '\u{202e}', '\u{2066}', '\u{2067}',
-                '\u{2068}', '\u{2069}',
-            ]
-            .contains(&c),
+            Self::Bidi => BIDI_CHARACTERS.contains(&c),
             Self::Block(range) => range.contains(&c),
             Self::Anything => true,
         }