search

mooman219 / fontdue

The fastest font renderer in the world, written in pure rust.
Apache License 2.0
1.38k stars 70 forks source link

Make hashbrown optional #153

Closed not-fl3 closed 1 month ago

not-fl3 commented 1 month ago

hashbrown became a fairly heavy dependency: 

├── fontdue v0.7.3
│   ├── hashbrown v0.13.2
│   │   └── ahash v0.8.11
│   │       ├── cfg-if v1.0.0
│   │       ├── once_cell v1.19.0
│   │       └── zerocopy v0.7.34
│   │           └── zerocopy-derive v0.7.34 (proc-macro)
│   │               ├── proc-macro2 v1.0.82
│   │               │   └── unicode-ident v1.0.12
│   │               ├── quote v1.0.36
│   │               │   └── proc-macro2 v1.0.82
│   │               │       └── unicode-ident v1.0.12
│   │               └── syn v2.0.63
│   │                   ├── proc-macro2 v1.0.82
│   │                   │   └── unicode-ident v1.0.12
│   │                   ├── quote v1.0.36
│   │                   │   └── proc-macro2 v1.0.82
│   │                   │       └── unicode-ident v1.0.12
│   │                   └── unicode-ident v1.0.12
│   │       [build-dependencies]
│   │       └── version_check v0.9.4
│   └── ttf-parser v0.15.2

hashbrown had the same problem in the past: https://github.com/tkaitchuck/aHash/pull/18), I believe it might be fixed again (I should do a PR there too!).

But so far - it would be nice to just fallback to std.

mooman219 commented 1 month ago

ah yeah that's unfortunate, sounds good to make it optional