Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Make all clippy-suggested functions const #296

Merged
merged 3 commits into from
May 20, 2022

Conversation

clubby789
Copy link
Contributor

Addresses #237

@clubby789
Copy link
Contributor Author

Tested with cargo +1.54 clippy -- -W clippy::missing_const_for_fn

@wtfsck
Copy link
Member

wtfsck commented May 20, 2022

Build failed, some of them can't be const. Try with mvex feature enabled too

cargo +1.54 clippy --features "mvex" -- -W clippy::missing_const_for_fn

@clubby789
Copy link
Contributor Author

Segfault in rustc? 🤔

@wtfsck
Copy link
Member

wtfsck commented May 20, 2022

Yes that's weird. I've restarted it, let's see if it still segfaults.

@clubby789
Copy link
Contributor Author

It happens locally for me too, and I can see some similar issues related to segfaults while compiling to wasm. Maybe an issue with llvm + const?

@wtfsck
Copy link
Member

wtfsck commented May 20, 2022

Yeah probably. Will see if I have time later this weekend to figure out which 'const' is the one triggering this segfault.

@wtfsck
Copy link
Member

wtfsck commented May 20, 2022

BTW, could you reproduce the crash with 1.54 or did you only try the latest version?

@clubby789
Copy link
Contributor Author

It only happens under nightly for me, 1.54 and stable both compile fine

@wtfsck
Copy link
Member

wtfsck commented May 20, 2022

Maybe your stable isn't 1.61, because that's what CI is using and it crashed both times.

@clubby789
Copy link
Contributor Author

My stable is 1.60, so I think that's correct. I've filed an issue about the crash at rust-lang/rust#97222

@wtfsck
Copy link
Member

wtfsck commented May 20, 2022

Seems like this crash is unrelated to your PR. I've created a new PR to use an older version of rustc when building the JS stuff, if that builds successfully I'll merge this.

@wtfsck wtfsck merged commit 8cccfc9 into icedland:master May 20, 2022
@wtfsck
Copy link
Member

wtfsck commented May 20, 2022

Thanks, merged it!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants