Skip to content
This repository was archived by the owner on Jan 29, 2025. It is now read-only.

[spv-out] Use IndexSet instead of HashSet for iterated sets (capabilities/extensions).#2389

Merged
jimblandy merged 1 commit intogfx-rs:masterfrom
EmbarkStudios:do-not-iterate-hashsets
Jun 21, 2023
Merged

[spv-out] Use IndexSet instead of HashSet for iterated sets (capabilities/extensions).#2389
jimblandy merged 1 commit intogfx-rs:masterfrom
EmbarkStudios:do-not-iterate-hashsets

Conversation

@eddyb
Copy link
Copy Markdown
Contributor

@eddyb eddyb commented Jun 21, 2023

Naga currently doesn't pass its own CI because its tests hardcode a specific hashbrown+fx iteration order.

And then upstream upgraded hashbrown in nightly-2023-06-16 which changed the order:

What's worse is I'm hitting this on a v0.10 backport, where I would prefer not to backport this PR too:

Copy link
Copy Markdown
Member

@jimblandy jimblandy left a comment

Choose a reason for hiding this comment

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

Looks good to me.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants