Move harfbuzz-traits implementations into component crates

Author: robertbastian

CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## Unreleased
+
+- Utils
+  - Retire the `icu_harfbuzz` crate. The `icu_properties` and `icu_normalizer` types now directly implement the `harfbuzz-traits`
+
 ## icu4x 2.1
 
 - Components

CODEOWNERS

@@ -27,7 +27,6 @@ components/segmenter/             @aethanyc @makotokato @sffc
 components/time/              @nekevss @robertbastian @sffc
 ffi/capi/                         @Manishearth
 ffi/ecma402/                      @filmil
-ffi/harfbuzz/                     @hsivonen
 provider/blob/                    @sffc @Manishearth
 provider/core/                    @sffc @Manishearth
 provider/source/                  @sffc @robertbastian @Manishearth

Cargo.lock

@@ -34,7 +34,6 @@ members = [
     "ffi/capi",
     "ffi/ecma402",
     "ffi/freertos",
-    "ffi/harfbuzz",
 
     # Provider
     "provider/adapters",
@@ -158,7 +157,6 @@ icu_time = { version = "~2.1.1", path = "components/time", default-features = fa
 icu_capi = { version = "~2.1.1", path = "ffi/capi", default-features = false }
 # icu4x_ecma402 never used as a dep
 # icu_freertos never used as a dep
-# icu_harfbuzz never used as a dep
 
 # Provider
 icu_provider_export = { version = "~2.1.1", path = "provider/export", default-features = false }
@@ -245,6 +243,7 @@ arrayvec = { version = "0.7.2", default-features = false }
 core_maths = { version = "0.1.0", default-features = false }
 displaydoc = { version = "0.2.3", default-features = false }
 either = { version = "1.9.0", default-features = false }
+harfbuzz-traits = { version = "0.6.0", default-features = false }
 libc_alloc = { version = "1.0.6", default-features = false }
 log = { version = "0.4.17", default-features = false }
 memchr = { version = "2.6.0", default-features = false }

Cargo.toml

@@ -20,6 +20,7 @@ license.workspace = true
 all-features = true
 
 [dependencies]
+harfbuzz-traits = { workspace = true, optional = true }
 icu_collections = { workspace = true }
 icu_properties = { workspace = true, optional = true }
 icu_provider = { workspace = true }
@@ -58,6 +59,7 @@ icu_properties = ["dep:icu_properties"]
 utf16_iter = ["dep:utf16_iter", "dep:write16"]
 # For dealing with potentially ill-formed UTF8 strings
 utf8_iter = ["dep:utf8_iter"]
+harfbuzz_traits = ["dep:harfbuzz-traits"]
 
 # added by accident
 write16 = []

components/normalizer/Cargo.toml

@@ -0,0 +1,68 @@
+// This file is part of ICU4X. For terms of use, please see the file
+// called LICENSE at the top level of the ICU4X source tree
+// (online at: https://github.com/unicode-org/icu4x/blob/main/LICENSE ).
+
+use crate::properties::{
+    CanonicalCombiningClassMap, CanonicalCombiningClassMapBorrowed, CanonicalComposition,
+    CanonicalCompositionBorrowed, CanonicalDecomposition, CanonicalDecompositionBorrowed,
+    Decomposed,
+};
+use harfbuzz_traits::{CombiningClassFunc, ComposeFunc, DecomposeFunc};
+
+impl ComposeFunc for CanonicalCompositionBorrowed<'_> {
+    fn compose(&self, a: char, b: char) -> Option<char> {
+        CanonicalCompositionBorrowed::compose(*self, a, b)
+    }
+}
+
+impl ComposeFunc for CanonicalComposition {
+    fn compose(&self, a: char, b: char) -> Option<char> {
+        ComposeFunc::compose(&self.as_borrowed(), a, b)
+    }
+}
+
+impl ComposeFunc for &'_ CanonicalComposition {
+    fn compose(&self, a: char, b: char) -> Option<char> {
+        ComposeFunc::compose(&self.as_borrowed(), a, b)
+    }
+}
+
+impl DecomposeFunc for CanonicalDecompositionBorrowed<'_> {
+    fn decompose(&self, ab: char) -> Option<(char, char)> {
+        match CanonicalDecompositionBorrowed::decompose(self, ab) {
+            Decomposed::Default => None,
+            Decomposed::Expansion(first, second) => Some((first, second)),
+            Decomposed::Singleton(single) => Some((single, '\0')),
+        }
+    }
+}
+
+impl DecomposeFunc for CanonicalDecomposition {
+    fn decompose(&self, ab: char) -> Option<(char, char)> {
+        DecomposeFunc::decompose(&self.as_borrowed(), ab)
+    }
+}
+
+impl DecomposeFunc for &'_ CanonicalDecomposition {
+    fn decompose(&self, ab: char) -> Option<(char, char)> {
+        DecomposeFunc::decompose(&self.as_borrowed(), ab)
+    }
+}
+
+impl CombiningClassFunc for CanonicalCombiningClassMapBorrowed<'_> {
+    fn combining_class(&self, ch: char) -> u8 {
+        self.get_u8(ch)
+    }
+}
+
+impl CombiningClassFunc for CanonicalCombiningClassMap {
+    fn combining_class(&self, ch: char) -> u8 {
+        CombiningClassFunc::combining_class(&self.as_borrowed(), ch)
+    }
+}
+
+impl CombiningClassFunc for &'_ CanonicalCombiningClassMap {
+    fn combining_class(&self, ch: char) -> u8 {
+        CombiningClassFunc::combining_class(&self.as_borrowed(), ch)
+    }
+}

components/normalizer/README.md

@@ -39,8 +39,10 @@
 //!
 //! The `properties` module provides the non-recursive canonical decomposition operation on a per `char` basis and
 //! the canonical compositon operation given two `char`s. It also provides access to the Canonical Combining Class
-//! property. These operations are primarily meant for [HarfBuzz](https://harfbuzz.github.io/) via the
-//! [`icu_harfbuzz`](https://docs.rs/icu_harfbuzz/latest/icu_harfbuzz/) crate.
+//! property. These operations are primarily meant for [HarfBuzz](https://harfbuzz.github.io/), the types
+//! [`CanonicalComposition`](properties::CanonicalComposition), [`CanonicalDecomposition`](properties::CanonicalDecomposition),
+//! and [`CanonicalCombiningClassMap`](properties::CanonicalCombiningClassMap) implement the [`harfbuzz_traits`] if
+//! the `harfbuzz_traits` Cargo feature is enabled.
 //!
 //! Notably, this normalizer does _not_ provide the normalization “quick check” that can result in “maybe” in
 //! addition to “yes” and “no”. The normalization checks provided by this crate always give a definitive
@@ -99,6 +101,8 @@ macro_rules! ccc {
     };
 }
 
+#[cfg(feature = "harfbuzz_traits")]
+mod harfbuzz;
 pub mod properties;
 pub mod provider;
 pub mod uts46;

components/normalizer/src/harfbuzz.rs

@@ -28,6 +28,7 @@ databake = { workspace = true, features = ["derive"], optional = true}
 serde = { workspace = true, features = ["derive"], optional = true }
 icu_locale_core = { workspace = true, features = ["zerovec"] }
 
+harfbuzz-traits = { workspace = true, optional = true }
 unicode-bidi = { workspace = true, optional = true }
 
 icu_properties_data = { workspace = true, optional = true }
@@ -39,6 +40,7 @@ icu = { path = "../../components/icu", default-features = false }
 default = ["compiled_data"]
 serde = ["dep:serde", "icu_locale_core/serde", "zerovec/serde", "icu_collections/serde", "icu_provider/serde", "zerotrie/serde"]
 datagen = ["serde", "dep:databake", "zerovec/databake", "icu_collections/databake", "icu_locale_core/databake", "zerotrie/databake", "icu_provider/export"]
-unicode_bidi = [ "dep:unicode-bidi" ]
+unicode_bidi = ["dep:unicode-bidi"]
+harfbuzz_traits = ["dep:harfbuzz-traits"]
 compiled_data = ["dep:icu_properties_data", "icu_provider/baked"]
 alloc = ["zerovec/alloc", "icu_collections/alloc", "serde?/alloc"]