[INFO] cloning repository https://github.com/wowemulation-dev/recast-rs
[INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/wowemulation-dev/recast-rs" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fwowemulation-dev%2Frecast-rs", kill_on_drop: false }`
[INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fwowemulation-dev%2Frecast-rs'...
[INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }`
[INFO] [stdout] b90f3730adc69732d37d4cff821a7db87a596fc1
[INFO] checking wowemulation-dev/recast-rs against a5689b034ad287112f709693384a37f1377eb9cc for pr-129249-3
[INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fwowemulation-dev%2Frecast-rs" "/workspace/builds/worker-5-tc2/source", kill_on_drop: false }`
[INFO] [stderr] Cloning into '/workspace/builds/worker-5-tc2/source'...
[INFO] [stderr] done.
[INFO] removed /workspace/builds/worker-5-tc2/source/.cargo/config.toml
[INFO] removed /workspace/builds/worker-5-tc2/source/rust-toolchain.toml
[INFO] started tweaking git repo https://github.com/wowemulation-dev/recast-rs
[INFO] finished tweaking git repo https://github.com/wowemulation-dev/recast-rs
[INFO] tweaked toml for git repo https://github.com/wowemulation-dev/recast-rs written to /workspace/builds/worker-5-tc2/source/Cargo.toml
[INFO] validating manifest of git repo https://github.com/wowemulation-dev/recast-rs on toolchain a5689b034ad287112f709693384a37f1377eb9cc
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }`
[INFO] crate git repo https://github.com/wowemulation-dev/recast-rs already has a lockfile, it will not be regenerated
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] [stderr] warning: unused workspace dependency `waymark-dynamic`
[INFO] [stderr]   --> Cargo.toml:72:25
[INFO] [stderr]    |
[INFO] [stderr] 72 | [workspace.dependencies.waymark-dynamic]
[INFO] [stderr]    |                         ^^^^^^^^^^^^^^^
[INFO] [stderr]    |
[INFO] [stderr]    = note: `cargo::unused_workspace_dependencies` is set to `warn` by default
[INFO] [stderr] help: consider removing the workspace dependency `waymark-dynamic`
[INFO] [stderr] warning: unused field `categories` in `workspace.package`
[INFO] [stderr]    --> Cargo.toml:237:1
[INFO] [stderr]     |
[INFO] [stderr] 237 | categories = ["game-development", "algorithms"]
[INFO] [stderr]     | ^^^^^^^^^^
[INFO] [stderr]     |
[INFO] [stderr]     = note: `cargo::unused_workspace_package_fields` is set to `warn` by default
[INFO] [stderr] help: consider removing the field `workspace.package.categories`
[INFO] [stderr] warning: unused field `keywords` in `workspace.package`
[INFO] [stderr]    --> Cargo.toml:239:1
[INFO] [stderr]     |
[INFO] [stderr] 239 | keywords = ["game", "pathfinding", "navmesh", "landmark", "waymark"]
[INFO] [stderr]     | ^^^^^^^^
[INFO] [stderr]     |
[INFO] [stderr] help: consider removing the field `workspace.package.keywords`
[INFO] [stderr] warning: workspace (manifest) generated 3 warnings
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-m" "1610612736" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:77db811e55d90add9212f6832c23229e3de5f1c1f9905e8cb07319ea15df73ac" "sleep" "infinity", kill_on_drop: false }`
[INFO] [stdout] 20f98c8e1c34c159f59c7855fbdfe9ed26e702c34882a9a09c449ba32a2b5de5
[INFO] running `Command { std: "docker" "start" "20f98c8e1c34c159f59c7855fbdfe9ed26e702c34882a9a09c449ba32a2b5de5", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "20f98c8e1c34c159f59c7855fbdfe9ed26e702c34882a9a09c449ba32a2b5de5", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "exec" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-w" "/opt/rustwide/workdir" "--user" "0:0" "20f98c8e1c34c159f59c7855fbdfe9ed26e702c34882a9a09c449ba32a2b5de5" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "20f98c8e1c34c159f59c7855fbdfe9ed26e702c34882a9a09c449ba32a2b5de5", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "exec" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid" "-e" "RUSTDOCFLAGS=--cap-lints=forbid" "-w" "/opt/rustwide/workdir" "--user" "0:0" "20f98c8e1c34c159f59c7855fbdfe9ed26e702c34882a9a09c449ba32a2b5de5" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stderr] warning: unused workspace dependency `waymark-dynamic`
[INFO] [stderr]   --> Cargo.toml:72:25
[INFO] [stderr]    |
[INFO] [stderr] 72 | [workspace.dependencies.waymark-dynamic]
[INFO] [stderr]    |                         ^^^^^^^^^^^^^^^
[INFO] [stderr]    |
[INFO] [stderr]    = note: `cargo::unused_workspace_dependencies` is set to `warn` by default
[INFO] [stderr] help: consider removing the workspace dependency `waymark-dynamic`
[INFO] [stderr] warning: unused field `categories` in `workspace.package`
[INFO] [stderr]    --> Cargo.toml:237:1
[INFO] [stderr]     |
[INFO] [stderr] 237 | categories = ["game-development", "algorithms"]
[INFO] [stderr]     | ^^^^^^^^^^
[INFO] [stderr]     |
[INFO] [stderr]     = note: `cargo::unused_workspace_package_fields` is set to `warn` by default
[INFO] [stderr] help: consider removing the field `workspace.package.categories`
[INFO] [stderr] warning: unused field `keywords` in `workspace.package`
[INFO] [stderr]    --> Cargo.toml:239:1
[INFO] [stderr]     |
[INFO] [stderr] 239 | keywords = ["game", "pathfinding", "navmesh", "landmark", "waymark"]
[INFO] [stderr]     | ^^^^^^^^
[INFO] [stderr]     |
[INFO] [stderr] help: consider removing the field `workspace.package.keywords`
[INFO] [stderr] warning: workspace (manifest) generated 3 warnings
[INFO] [stderr]    Compiling zmij v1.0.15
[INFO] [stderr]    Compiling heapless v0.7.17
[INFO] [stderr]     Checking spin v0.9.8
[INFO] [stderr]     Checking clap_lex v0.7.7
[INFO] [stderr]    Compiling syn v2.0.114
[INFO] [stderr]     Checking glam v0.30.10
[INFO] [stderr]     Checking bitflags v2.10.0
[INFO] [stderr]     Checking assert_approx_eq v1.1.0
[INFO] [stderr]     Checking ordered-float v5.1.0
[INFO] [stderr]     Checking regex-syntax v0.8.9
[INFO] [stderr]     Checking itertools v0.10.5
[INFO] [stderr]     Checking plotters v0.3.7
[INFO] [stderr]     Checking is-terminal v0.4.17
[INFO] [stderr]     Checking lz4_flex v0.12.1
[INFO] [stderr]     Checking event-listener v5.4.1
[INFO] [stderr]     Checking parking_lot_core v0.9.12
[INFO] [stderr]     Checking futures-core v0.3.31
[INFO] [stderr]     Checking socket2 v0.6.2
[INFO] [stderr]     Checking futures-lite v2.6.1
[INFO] [stderr]     Checking clap_builder v4.5.54
[INFO] [stderr]     Checking parking_lot v0.12.5
[INFO] [stderr]     Checking event-listener-strategy v0.5.4
[INFO] [stderr]     Checking async-lock v3.4.2
[INFO] [stderr]     Checking rustix v1.1.3
[INFO] [stderr]     Checking serde_json v1.0.149
[INFO] [stderr]     Checking regex-automata v0.4.14
[INFO] [stderr]     Checking criterion-plot v0.5.0
[INFO] [stderr]     Checking tempfile v3.24.0
[INFO] [stderr]     Checking regex v1.12.3
[INFO] [stderr]    Compiling serde_derive v1.0.228
[INFO] [stderr]    Compiling thiserror-impl v2.0.18
[INFO] [stderr]    Compiling zerocopy-derive v0.8.39
[INFO] [stderr]    Compiling clap_derive v4.5.49
[INFO] [stderr]    Compiling tokio-macros v2.6.0
[INFO] [stderr]     Checking tokio v1.49.0
[INFO] [stderr]     Checking thiserror v2.0.18
[INFO] [stderr]     Checking cobs v0.3.0
[INFO] [stderr]     Checking zerocopy v0.8.39
[INFO] [stderr]     Checking clap v4.5.54
[INFO] [stderr]     Checking serde v1.0.228
[INFO] [stderr]     Checking tinytemplate v1.2.1
[INFO] [stderr]     Checking postcard v1.1.3
[INFO] [stderr]     Checking landmark-common v0.1.0 (/opt/rustwide/workdir/crates/landmark-common)
[INFO] [stderr]     Checking landmark v0.1.0 (/opt/rustwide/workdir/crates/landmark)
[INFO] [stdout] error: useless conversion to the same type: `BuildError`
[INFO] [stdout]    --> crates/landmark/src/heightfield_layers.rs:135:24
[INFO] [stdout]     |
[INFO] [stdout] 135 |             return Err(BuildError::SpanOutOfBounds { x, y: z }.into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout]     = note: `#[deny(self_type_conversion)]` on by default
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 135 -             return Err(BuildError::SpanOutOfBounds { x, y: z }.into());
[INFO] [stdout] 135 +             return Err(BuildError::SpanOutOfBounds { x, y: z });
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 135 -             return Err(BuildError::SpanOutOfBounds { x, y: z }.into());
[INFO] [stdout] 135 +             return Err(<BuildError as Into<_>>::into(BuildError::SpanOutOfBounds { x, y: z }));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `BuildError`
[INFO] [stdout]    --> crates/landmark/src/heightfield_layers.rs:319:36
[INFO] [stdout]     |
[INFO] [stdout] 319 |                         return Err(BuildError::RegionIdOverflow.into());
[INFO] [stdout]     |                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 319 -                         return Err(BuildError::RegionIdOverflow.into());
[INFO] [stdout] 319 +                         return Err(BuildError::RegionIdOverflow);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 319 -                         return Err(BuildError::RegionIdOverflow.into());
[INFO] [stdout] 319 +                         return Err(<BuildError as Into<_>>::into(BuildError::RegionIdOverflow));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `BuildError`
[INFO] [stdout]    --> crates/landmark/src/heightfield_layers.rs:501:44
[INFO] [stdout]     |
[INFO] [stdout] 501 | ...                   return Err(BuildError::LayerOverflow { x: 0, y: 0 }.into());
[INFO] [stdout]     |                                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 501 -                                 return Err(BuildError::LayerOverflow { x: 0, y: 0 }.into());
[INFO] [stdout] 501 +                                 return Err(BuildError::LayerOverflow { x: 0, y: 0 });
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 501 -                                 return Err(BuildError::LayerOverflow { x: 0, y: 0 }.into());
[INFO] [stdout] 501 +                                 return Err(<BuildError as Into<_>>::into(BuildError::LayerOverflow { x: 0, y: 0 }));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `BuildError`
[INFO] [stdout]    --> crates/landmark/src/heightfield_layers.rs:596:44
[INFO] [stdout]     |
[INFO] [stdout] 596 | ...                   return Err(BuildError::LayerOverflow { x: 0, y: 0 }.into());
[INFO] [stdout]     |                                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 596 -                                 return Err(BuildError::LayerOverflow { x: 0, y: 0 }.into());
[INFO] [stdout] 596 +                                 return Err(BuildError::LayerOverflow { x: 0, y: 0 });
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 596 -                                 return Err(BuildError::LayerOverflow { x: 0, y: 0 }.into());
[INFO] [stdout] 596 +                                 return Err(<BuildError as Into<_>>::into(BuildError::LayerOverflow { x: 0, y: 0 }));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `BuildError`
[INFO] [stdout]    --> crates/landmark/src/polymesh.rs:176:24
[INFO] [stdout]     |
[INFO] [stdout] 176 |               return Err(BuildError::TooManyVertices {
[INFO] [stdout]     |  ________________________^
[INFO] [stdout] 177 | |                 count: max_vertices,
[INFO] [stdout] 178 | |                 max: 0xfffe,
[INFO] [stdout] 179 | |             }
[INFO] [stdout] 180 | |             .into());
[INFO] [stdout]     | |___________________^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 179 -             }
[INFO] [stdout] 180 -             .into());
[INFO] [stdout] 179 +             });
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 176 ~             return Err(<BuildError as Into<_>>::into(BuildError::TooManyVertices {
[INFO] [stdout] 177 |                 count: max_vertices,
[INFO] [stdout] 178 |                 max: 0xfffe,
[INFO] [stdout] 179 ~             }));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `BuildError`
[INFO] [stdout]    --> crates/landmark/src/polymesh.rs:282:32
[INFO] [stdout]     |
[INFO] [stdout] 282 |                       return Err(BuildError::TooManyPolygons {
[INFO] [stdout]     |  ________________________________^
[INFO] [stdout] 283 | |                         count: mesh.npolys,
[INFO] [stdout] 284 | |                         max: max_tris,
[INFO] [stdout] 285 | |                     }
[INFO] [stdout] 286 | |                     .into());
[INFO] [stdout]     | |___________________________^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 285 -                     }
[INFO] [stdout] 286 -                     .into());
[INFO] [stdout] 285 +                     });
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 282 ~                     return Err(<BuildError as Into<_>>::into(BuildError::TooManyPolygons {
[INFO] [stdout] 283 |                         count: mesh.npolys,
[INFO] [stdout] 284 |                         max: max_tris,
[INFO] [stdout] 285 ~                     }));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `BuildError`
[INFO] [stdout]    --> crates/landmark/src/polymesh.rs:305:24
[INFO] [stdout]     |
[INFO] [stdout] 305 |               return Err(BuildError::TooManyVertices {
[INFO] [stdout]     |  ________________________^
[INFO] [stdout] 306 | |                 count: mesh.nverts,
[INFO] [stdout] 307 | |                 max: 0xffff,
[INFO] [stdout] 308 | |             }
[INFO] [stdout] 309 | |             .into());
[INFO] [stdout]     | |___________________^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 308 -             }
[INFO] [stdout] 309 -             .into());
[INFO] [stdout] 308 +             });
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 305 ~             return Err(<BuildError as Into<_>>::into(BuildError::TooManyVertices {
[INFO] [stdout] 306 |                 count: mesh.nverts,
[INFO] [stdout] 307 |                 max: 0xffff,
[INFO] [stdout] 308 ~             }));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `BuildError`
[INFO] [stdout]    --> crates/landmark/src/polymesh.rs:312:24
[INFO] [stdout]     |
[INFO] [stdout] 312 |               return Err(BuildError::TooManyPolygons {
[INFO] [stdout]     |  ________________________^
[INFO] [stdout] 313 | |                 count: mesh.npolys,
[INFO] [stdout] 314 | |                 max: 0xffff,
[INFO] [stdout] 315 | |             }
[INFO] [stdout] 316 | |             .into());
[INFO] [stdout]     | |___________________^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 315 -             }
[INFO] [stdout] 316 -             .into());
[INFO] [stdout] 315 +             });
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 312 ~             return Err(<BuildError as Into<_>>::into(BuildError::TooManyPolygons {
[INFO] [stdout] 313 |                 count: mesh.npolys,
[INFO] [stdout] 314 |                 max: 0xffff,
[INFO] [stdout] 315 ~             }));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `BuildError`
[INFO] [stdout]    --> crates/landmark/src/watershed.rs:495:32
[INFO] [stdout]     |
[INFO] [stdout] 495 |                     return Err(BuildError::RegionIdOverflow.into());
[INFO] [stdout]     |                                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 495 -                     return Err(BuildError::RegionIdOverflow.into());
[INFO] [stdout] 495 +                     return Err(BuildError::RegionIdOverflow);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 495 -                     return Err(BuildError::RegionIdOverflow.into());
[INFO] [stdout] 495 +                     return Err(<BuildError as Into<_>>::into(BuildError::RegionIdOverflow));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `BuildError`
[INFO] [stdout]    --> crates/landmark/src/lib.rs:221:24
[INFO] [stdout]     |
[INFO] [stdout] 221 |             return Err(BuildError::VertexArrayNotTripled.into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 221 -             return Err(BuildError::VertexArrayNotTripled.into());
[INFO] [stdout] 221 +             return Err(BuildError::VertexArrayNotTripled);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 221 -             return Err(BuildError::VertexArrayNotTripled.into());
[INFO] [stdout] 221 +             return Err(<BuildError as Into<_>>::into(BuildError::VertexArrayNotTripled));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `BuildError`
[INFO] [stdout]    --> crates/landmark/src/lib.rs:225:24
[INFO] [stdout]     |
[INFO] [stdout] 225 |             return Err(BuildError::IndexArrayNotTripled.into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 225 -             return Err(BuildError::IndexArrayNotTripled.into());
[INFO] [stdout] 225 +             return Err(BuildError::IndexArrayNotTripled);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 225 -             return Err(BuildError::IndexArrayNotTripled.into());
[INFO] [stdout] 225 +             return Err(<BuildError as Into<_>>::into(BuildError::IndexArrayNotTripled));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `BuildError`
[INFO] [stdout]    --> crates/landmark/src/lib.rs:243:28
[INFO] [stdout]     |
[INFO] [stdout] 243 |                   return Err(BuildError::TriangleIndexOutOfBounds {
[INFO] [stdout]     |  ____________________________^
[INFO] [stdout] 244 | |                     i0: idx0,
[INFO] [stdout] 245 | |                     i1: idx1,
[INFO] [stdout] 246 | |                     i2: idx2,
[INFO] [stdout] ...   |
[INFO] [stdout] 249 | |                 .into());
[INFO] [stdout]     | |_______________________^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 248 -                 }
[INFO] [stdout] 249 -                 .into());
[INFO] [stdout] 248 +                 });
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 243 ~                 return Err(<BuildError as Into<_>>::into(BuildError::TriangleIndexOutOfBounds {
[INFO] [stdout] 244 |                     i0: idx0,
[INFO] [stdout] ...
[INFO] [stdout] 247 |                     max: max_idx - 1,
[INFO] [stdout] 248 ~                 }));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `landmark` (lib) due to 12 previous errors
[INFO] [stderr] warning: build failed, waiting for other jobs to finish...
[INFO] [stderr] warning: unused dependency `log`
[INFO] [stderr]   --> crates/landmark-common/Cargo.toml:17:1
[INFO] [stderr]    |
[INFO] [stderr] 17 | log = { workspace = true }
[INFO] [stderr]    | ^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stderr]    |
[INFO] [stderr]    = note: `cargo::unused_dependencies` is set to `warn` by default
[INFO] [stderr] help: consider removing the dependency on `log`
[INFO] [stderr] warning: `landmark-common` (manifest) generated 1 warning
[INFO] running `Command { std: "docker" "inspect" "20f98c8e1c34c159f59c7855fbdfe9ed26e702c34882a9a09c449ba32a2b5de5", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "20f98c8e1c34c159f59c7855fbdfe9ed26e702c34882a9a09c449ba32a2b5de5", kill_on_drop: false }`
[INFO] [stdout] 20f98c8e1c34c159f59c7855fbdfe9ed26e702c34882a9a09c449ba32a2b5de5
