Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

ncollide3d is missing the Fn trait compilation failure. #79

Open
Natim opened this issue May 10, 2019 · 1 comment · May be fixed by #80
Open

ncollide3d is missing the Fn trait compilation failure. #79

Natim opened this issue May 10, 2019 · 1 comment · May be fixed by #80

Comments

@Natim
Copy link

Natim commented May 10, 2019

Describe the bug

When trying to compile with the last rust.

To Reproduce

cargo run --release
error[E0525]: expected a closure that implements the `Fn` trait, but this closure only implements `FnMut`
   --> ~/.cargo/registry/src/github.com-1ecc6299db9ec823/ncollide3d-0.18.1/src/transformation/hacd.rs:745:47
    |
745 |         let mut add_triangle_edges = Box::new(|i: usize, t: &Point3<u32>| {
    |                                               ^^^^^^^^^^^^^^^^^^^^^^^^^^^ this closure implements `FnMut`, not `Fn`
...
749 |                 let other = match prim_edges.entry(*e) {
    |                                   ---------- closure is `FnMut` because it mutates the variable `prim_edges` here
...
765 |                     add_triangle_edges(i, t)
    |                     ------------------------ the requirement to implement `Fn` derives from here

error: aborting due to previous error
@Natim
Copy link
Author

Natim commented May 14, 2019

Refs dimforge/ncollide#289

@Natim Natim linked a pull request May 14, 2019 that will close this issue
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 a pull request may close this issue.

1 participant