Skip to content

Upgrade phf to 0.14#433

Open
nicoburns wants to merge 1 commit into
servo:mainfrom
nicoburns:phf014
Open

Upgrade phf to 0.14#433
nicoburns wants to merge 1 commit into
servo:mainfrom
nicoburns:phf014

Conversation

@nicoburns

Copy link
Copy Markdown
Contributor

There is potentially a decent speed boost available behind the experimental new ptrhash feature (see: rust-phf/rust-phf#392). But that is not enabled here.

Signed-off-by: Nico Burns <nico@nicoburns.com>
@mrobinson

Copy link
Copy Markdown
Member
Run cargo fmt --check && cargo clippy -- -Dwarnings
    Updating crates.io index
 Downloading crates ...
  Downloaded phf_macros v0.14.0
error: failed to download replaced source registry `crates-io`

Caused by:
  failed to parse manifest at `/home/runner/.cargo/registry/src/index.crates.io-6f17d22bba15001f/phf_macros-0.14.0/Cargo.toml`

Caused by:
  failed to parse the `edition` key

Caused by:
  this version of Cargo is older than the `2024` edition, and only supports `2015`, `2018`, and `2021` editions.
Error: Process completed with exit code 101.

@emilio

emilio commented Jul 22, 2026

Copy link
Copy Markdown
Member

Bumping the msrv might be acceptable.

Comment thread Cargo.toml
cssparser-macros = { path = "./macros", version = "0.7.0", optional = true }
malloc_size_of = { version = "0.1", default-features = false, optional = true }
phf = { version = "0.13.1", features = ["macros"], optional = true }
phf = { version = "0.14.0", features = ["macros"], optional = true }

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also if this now doesn't require syn 2.0, let's just update to syn 3.0 exclusively while at it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants