Skip to content

Change AllocScqRing's MODE from bool to usize#5

Open
khyperia wants to merge 1 commit intoDiscordJim:mainfrom
khyperia:allocscqring-usize
Open

Change AllocScqRing's MODE from bool to usize#5
khyperia wants to merge 1 commit intoDiscordJim:mainfrom
khyperia:allocscqring-usize

Conversation

@khyperia
Copy link

Hi! Unfortunately, there is a breaking change happening soon in the Rust compiler. Generics in type aliases were not previously fully checked, causing type errors to not be reported properly. You can read more about the change here.

It seems like the AllocScqRing type alias has a slight typo in it, the ScqRing type takes a usize MODE, but AllocScqRing has a bool parameter. This will cause a compiler error in future Rust versions.

Sorry for the hassle! If you have questions, feel free to ask me here, or reach out to me or the rust team on the rust-lang Zulip chat.

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.

1 participant