[INFO] fetching crate isohypse-tsg 0.12.0...
[INFO] checking isohypse-tsg-0.12.0 against a5689b034ad287112f709693384a37f1377eb9cc for pr-129249-3
[INFO] extracting crate isohypse-tsg 0.12.0 into /workspace/builds/worker-5-tc2/source
[INFO] started tweaking crates.io crate isohypse-tsg 0.12.0
[INFO] finished tweaking crates.io crate isohypse-tsg 0.12.0
[INFO] tweaked toml for crates.io crate isohypse-tsg 0.12.0 written to /workspace/builds/worker-5-tc2/source/Cargo.toml
[INFO] validating manifest of crates.io crate isohypse-tsg 0.12.0 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 crates.io crate isohypse-tsg 0.12.0 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: `package.homepage` is redundant with `package.repository`
[INFO] [stderr]   --> Cargo.toml:83:12
[INFO] [stderr]    |
[INFO] [stderr] 83 | homepage = "https://github.com/tree-sitter/tree-sitter-graph/"
[INFO] [stderr]    |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stderr] ...
[INFO] [stderr] 87 | repository = "https://github.com/tree-sitter/tree-sitter-graph/"
[INFO] [stderr]    |              ---------------------------------------------------
[INFO] [stderr]    |
[INFO] [stderr]    = note: `cargo::redundant_homepage` is set to `warn` by default
[INFO] [stderr] help: consider removing `package.homepage`
[INFO] [stderr] warning: `isohypse-tsg` (manifest) generated 1 warning
[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] 38778ff46bbf6fddef3961c0ef9b969bb61f88925374390514258951736eef3b
[INFO] running `Command { std: "docker" "start" "38778ff46bbf6fddef3961c0ef9b969bb61f88925374390514258951736eef3b", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "38778ff46bbf6fddef3961c0ef9b969bb61f88925374390514258951736eef3b", 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" "38778ff46bbf6fddef3961c0ef9b969bb61f88925374390514258951736eef3b" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "38778ff46bbf6fddef3961c0ef9b969bb61f88925374390514258951736eef3b", 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" "38778ff46bbf6fddef3961c0ef9b969bb61f88925374390514258951736eef3b" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stderr] warning: `package.homepage` is redundant with `package.repository`
[INFO] [stderr]   --> Cargo.toml:83:12
[INFO] [stderr]    |
[INFO] [stderr] 83 | homepage = "https://github.com/tree-sitter/tree-sitter-graph/"
[INFO] [stderr]    |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stderr] ...
[INFO] [stderr] 87 | repository = "https://github.com/tree-sitter/tree-sitter-graph/"
[INFO] [stderr]    |              ---------------------------------------------------
[INFO] [stderr]    |
[INFO] [stderr]    = note: `cargo::redundant_homepage` is set to `warn` by default
[INFO] [stderr] help: consider removing `package.homepage`
[INFO] [stderr] warning: `isohypse-tsg` (manifest) generated 1 warning
[INFO] [stderr]    Compiling find-msvc-tools v0.1.11
[INFO] [stderr]    Compiling serde_json v1.0.151
[INFO] [stderr]    Compiling proc-macro2 v1.0.92
[INFO] [stderr]    Compiling unicode-ident v1.0.14
[INFO] [stderr]    Compiling serde_core v1.0.229
[INFO] [stderr]    Compiling itoa v1.0.14
[INFO] [stderr]    Compiling libc v0.2.189
[INFO] [stderr]     Checking tree-sitter-language v0.1.3
[INFO] [stderr]     Checking aho-corasick v1.1.3
[INFO] [stderr]    Compiling indexmap v2.7.0
[INFO] [stderr]     Checking hashbrown v0.15.2
[INFO] [stderr]     Checking log v0.4.22
[INFO] [stderr]     Checking streaming-iterator v0.1.9
[INFO] [stderr]     Checking smallvec v1.13.2
[INFO] [stderr]     Checking humantime v2.1.0
[INFO] [stderr]    Compiling indoc v1.0.9
[INFO] [stderr]    Compiling cc v1.4.4
[INFO] [stderr]    Compiling quote v1.0.37
[INFO] [stderr]    Compiling syn v2.0.90
[INFO] [stderr]     Checking regex-automata v0.4.9
[INFO] [stderr]     Checking atty v0.2.14
[INFO] [stderr]    Compiling tree-sitter-python v0.23.5
[INFO] [stderr]     Checking regex v1.11.1
[INFO] [stderr]     Checking env_logger v0.9.3
[INFO] [stderr]    Compiling thiserror-impl v1.0.69
[INFO] [stderr]     Checking thiserror v1.0.69
[INFO] [stderr]    Compiling tree-sitter v0.25.10
[INFO] [stderr]     Checking isohypse-tsg v0.12.0 (/opt/rustwide/workdir)
[INFO] [stdout] error: useless conversion to the same type: `StringConstant`
[INFO] [stdout]    --> src/ast.rs:786:36
[INFO] [stdout]     |
[INFO] [stdout] 786 |         Expression::StringConstant(StringConstant { value }.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] 786 -         Expression::StringConstant(StringConstant { value }.into())
[INFO] [stdout] 786 +         Expression::StringConstant(StringConstant { value })
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 786 -         Expression::StringConstant(StringConstant { value }.into())
[INFO] [stdout] 786 +         Expression::StringConstant(<StringConstant as Into<_>>::into(StringConstant { value }))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `graph::Value`
[INFO] [stdout]    --> src/execution/strict.rs:663:12
[INFO] [stdout]     |
[INFO] [stdout] 663 |           Ok(Value::from_nodes(
[INFO] [stdout]     |  ____________^
[INFO] [stdout] 664 | |             exec.graph,
[INFO] [stdout] 665 | |             exec.mat
[INFO] [stdout] 666 | |                 .nodes_for_capture_index(self.stanza_capture_index as u32),
[INFO] [stdout] ...   |
[INFO] [stdout] 669 | |         .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] 668 -         )
[INFO] [stdout] 669 -         .into())
[INFO] [stdout] 668 +         ))
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 663 ~         Ok(<graph::Value as Into<_>>::into(Value::from_nodes(
[INFO] [stdout] 664 |             exec.graph,
[INFO] [stdout] ...
[INFO] [stdout] 667 |             self.quantifier,
[INFO] [stdout] 668 ~         )))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `graph::Value`
[INFO] [stdout]    --> src/execution.rs:338:25
[INFO] [stdout]     |
[INFO] [stdout] 338 |                 None => Value::Null.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] 338 -                 None => Value::Null.into(),
[INFO] [stdout] 338 +                 None => Value::Null,
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 338 -                 None => Value::Null.into(),
[INFO] [stdout] 338 +                 None => <graph::Value as Into<_>>::into(Value::Null),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `graph::Value`
[INFO] [stdout]    --> src/graph.rs:283:34
[INFO] [stdout]     |
[INFO] [stdout] 283 |                     Err(o.insert(value.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] 283 -                     Err(o.insert(value.into()))
[INFO] [stdout] 283 +                     Err(o.insert(value))
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 283 -                     Err(o.insert(value.into()))
[INFO] [stdout] 283 +                     Err(o.insert(<graph::Value as Into<_>>::into(value)))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `std::string::String`
[INFO] [stdout]    --> src/parser.rs:603:59
[INFO] [stdout]     |
[INFO] [stdout] 603 |                     .map_err(|_| ParseError::InvalidRegex(pattern.into(), pattern_location))?;
[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] 603 -                     .map_err(|_| ParseError::InvalidRegex(pattern.into(), pattern_location))?;
[INFO] [stdout] 603 +                     .map_err(|_| ParseError::InvalidRegex(pattern, pattern_location))?;
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 603 -                     .map_err(|_| ParseError::InvalidRegex(pattern.into(), pattern_location))?;
[INFO] [stdout] 603 +                     .map_err(|_| ParseError::InvalidRegex(<std::string::String as Into<_>>::into(pattern), pattern_location))?;
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `ast::Capture`
[INFO] [stdout]    --> src/parser.rs:932:12
[INFO] [stdout]     |
[INFO] [stdout] 932 |           Ok(ast::Capture {
[INFO] [stdout]     |  ____________^
[INFO] [stdout] 933 | |             name,
[INFO] [stdout] 934 | |             quantifier: Zero,                 // set in checker
[INFO] [stdout] 935 | |             file_capture_index: usize::MAX,   // set in checker
[INFO] [stdout] ...   |
[INFO] [stdout] 939 | |         .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] 938 -         }
[INFO] [stdout] 939 -         .into())
[INFO] [stdout] 938 +         })
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 932 ~         Ok(<ast::Capture as Into<_>>::into(ast::Capture {
[INFO] [stdout] 933 |             name,
[INFO] [stdout] ...
[INFO] [stdout] 937 |             location,
[INFO] [stdout] 938 ~         }))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `RegexCapture`
[INFO] [stdout]    --> src/parser.rs:979:12
[INFO] [stdout]     |
[INFO] [stdout] 979 |         Ok(ast::RegexCapture { match_index }.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] 979 -         Ok(ast::RegexCapture { match_index }.into())
[INFO] [stdout] 979 +         Ok(ast::RegexCapture { match_index })
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 979 -         Ok(ast::RegexCapture { match_index }.into())
[INFO] [stdout] 979 +         Ok(<RegexCapture as Into<_>>::into(ast::RegexCapture { match_index }))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]   --> src/parse_error.rs:28:24
[INFO] [stdout]    |
[INFO] [stdout] 28 |     pub fn first(tree: &Tree) -> Option<ParseError> {
[INFO] [stdout]    |                        ^^^^^            ^^^^^^^^^^ the same lifetime is hidden here
[INFO] [stdout]    |                        |
[INFO] [stdout]    |                        the lifetime is elided here
[INFO] [stdout]    |
[INFO] [stdout]    = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]    = note: `#[warn(mismatched_lifetime_syntaxes)]` on by default
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]    |
[INFO] [stdout] 28 |     pub fn first(tree: &Tree) -> Option<ParseError<'_>> {
[INFO] [stdout]    |                                                   ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/parse_error.rs:245:18
[INFO] [stdout]     |
[INFO] [stdout] 245 |     pub fn error(&self) -> &ParseError {
[INFO] [stdout]     |                  ^^^^^     ^^^^^^^^^^^
[INFO] [stdout]     |                  |         ||
[INFO] [stdout]     |                  |         |the same lifetime is hidden here
[INFO] [stdout]     |                  |         the same lifetime is elided here
[INFO] [stdout]     |                  the lifetime is elided here
[INFO] [stdout]     |
[INFO] [stdout]     = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]     |
[INFO] [stdout] 245 |     pub fn error(&self) -> &ParseError<'_> {
[INFO] [stdout]     |                                       ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/parse_error.rs:288:18
[INFO] [stdout]     |
[INFO] [stdout] 288 |     pub fn error(&self) -> &Option<ParseError> {
[INFO] [stdout]     |                  ^^^^^     ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |                  |         |       |
[INFO] [stdout]     |                  |         |       the same lifetime is hidden here
[INFO] [stdout]     |                  |         the same lifetime is elided here
[INFO] [stdout]     |                  the lifetime is elided here
[INFO] [stdout]     |
[INFO] [stdout]     = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]     |
[INFO] [stdout] 288 |     pub fn error(&self) -> &Option<ParseError<'_>> {
[INFO] [stdout]     |                                              ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/parse_error.rs:341:19
[INFO] [stdout]     |
[INFO] [stdout] 341 |     pub fn errors(&self) -> &Vec<ParseError> {
[INFO] [stdout]     |                   ^^^^^     ^^^^^^^^^^^^^^^^
[INFO] [stdout]     |                   |         |    |
[INFO] [stdout]     |                   |         |    the same lifetime is hidden here
[INFO] [stdout]     |                   |         the same lifetime is elided here
[INFO] [stdout]     |                   the lifetime is elided here
[INFO] [stdout]     |
[INFO] [stdout]     = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]     |
[INFO] [stdout] 341 |     pub fn errors(&self) -> &Vec<ParseError<'_>> {
[INFO] [stdout]     |                                            ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `isohypse-tsg` (lib test) due to 7 previous errors; 4 warnings emitted
[INFO] [stderr] warning: build failed, waiting for other jobs to finish...
[INFO] [stdout] error: useless conversion to the same type: `StringConstant`
[INFO] [stdout]    --> src/ast.rs:786:36
[INFO] [stdout]     |
[INFO] [stdout] 786 |         Expression::StringConstant(StringConstant { value }.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] 786 -         Expression::StringConstant(StringConstant { value }.into())
[INFO] [stdout] 786 +         Expression::StringConstant(StringConstant { value })
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 786 -         Expression::StringConstant(StringConstant { value }.into())
[INFO] [stdout] 786 +         Expression::StringConstant(<StringConstant as Into<_>>::into(StringConstant { value }))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `graph::Value`
[INFO] [stdout]    --> src/execution/strict.rs:663:12
[INFO] [stdout]     |
[INFO] [stdout] 663 |           Ok(Value::from_nodes(
[INFO] [stdout]     |  ____________^
[INFO] [stdout] 664 | |             exec.graph,
[INFO] [stdout] 665 | |             exec.mat
[INFO] [stdout] 666 | |                 .nodes_for_capture_index(self.stanza_capture_index as u32),
[INFO] [stdout] ...   |
[INFO] [stdout] 669 | |         .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] 668 -         )
[INFO] [stdout] 669 -         .into())
[INFO] [stdout] 668 +         ))
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 663 ~         Ok(<graph::Value as Into<_>>::into(Value::from_nodes(
[INFO] [stdout] 664 |             exec.graph,
[INFO] [stdout] ...
[INFO] [stdout] 667 |             self.quantifier,
[INFO] [stdout] 668 ~         )))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `graph::Value`
[INFO] [stdout]    --> src/execution.rs:338:25
[INFO] [stdout]     |
[INFO] [stdout] 338 |                 None => Value::Null.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] 338 -                 None => Value::Null.into(),
[INFO] [stdout] 338 +                 None => Value::Null,
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 338 -                 None => Value::Null.into(),
[INFO] [stdout] 338 +                 None => <graph::Value as Into<_>>::into(Value::Null),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `graph::Value`
[INFO] [stdout]    --> src/graph.rs:283:34
[INFO] [stdout]     |
[INFO] [stdout] 283 |                     Err(o.insert(value.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] 283 -                     Err(o.insert(value.into()))
[INFO] [stdout] 283 +                     Err(o.insert(value))
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 283 -                     Err(o.insert(value.into()))
[INFO] [stdout] 283 +                     Err(o.insert(<graph::Value as Into<_>>::into(value)))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `std::string::String`
[INFO] [stdout]    --> src/parser.rs:603:59
[INFO] [stdout]     |
[INFO] [stdout] 603 |                     .map_err(|_| ParseError::InvalidRegex(pattern.into(), pattern_location))?;
[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] 603 -                     .map_err(|_| ParseError::InvalidRegex(pattern.into(), pattern_location))?;
[INFO] [stdout] 603 +                     .map_err(|_| ParseError::InvalidRegex(pattern, pattern_location))?;
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 603 -                     .map_err(|_| ParseError::InvalidRegex(pattern.into(), pattern_location))?;
[INFO] [stdout] 603 +                     .map_err(|_| ParseError::InvalidRegex(<std::string::String as Into<_>>::into(pattern), pattern_location))?;
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `ast::Capture`
[INFO] [stdout]    --> src/parser.rs:932:12
[INFO] [stdout]     |
[INFO] [stdout] 932 |           Ok(ast::Capture {
[INFO] [stdout]     |  ____________^
[INFO] [stdout] 933 | |             name,
[INFO] [stdout] 934 | |             quantifier: Zero,                 // set in checker
[INFO] [stdout] 935 | |             file_capture_index: usize::MAX,   // set in checker
[INFO] [stdout] ...   |
[INFO] [stdout] 939 | |         .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] 938 -         }
[INFO] [stdout] 939 -         .into())
[INFO] [stdout] 938 +         })
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 932 ~         Ok(<ast::Capture as Into<_>>::into(ast::Capture {
[INFO] [stdout] 933 |             name,
[INFO] [stdout] ...
[INFO] [stdout] 937 |             location,
[INFO] [stdout] 938 ~         }))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `RegexCapture`
[INFO] [stdout]    --> src/parser.rs:979:12
[INFO] [stdout]     |
[INFO] [stdout] 979 |         Ok(ast::RegexCapture { match_index }.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] 979 -         Ok(ast::RegexCapture { match_index }.into())
[INFO] [stdout] 979 +         Ok(ast::RegexCapture { match_index })
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 979 -         Ok(ast::RegexCapture { match_index }.into())
[INFO] [stdout] 979 +         Ok(<RegexCapture as Into<_>>::into(ast::RegexCapture { match_index }))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]   --> src/parse_error.rs:28:24
[INFO] [stdout]    |
[INFO] [stdout] 28 |     pub fn first(tree: &Tree) -> Option<ParseError> {
[INFO] [stdout]    |                        ^^^^^            ^^^^^^^^^^ the same lifetime is hidden here
[INFO] [stdout]    |                        |
[INFO] [stdout]    |                        the lifetime is elided here
[INFO] [stdout]    |
[INFO] [stdout]    = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]    = note: `#[warn(mismatched_lifetime_syntaxes)]` on by default
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]    |
[INFO] [stdout] 28 |     pub fn first(tree: &Tree) -> Option<ParseError<'_>> {
[INFO] [stdout]    |                                                   ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/parse_error.rs:245:18
[INFO] [stdout]     |
[INFO] [stdout] 245 |     pub fn error(&self) -> &ParseError {
[INFO] [stdout]     |                  ^^^^^     ^^^^^^^^^^^
[INFO] [stdout]     |                  |         ||
[INFO] [stdout]     |                  |         |the same lifetime is hidden here
[INFO] [stdout]     |                  |         the same lifetime is elided here
[INFO] [stdout]     |                  the lifetime is elided here
[INFO] [stdout]     |
[INFO] [stdout]     = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]     |
[INFO] [stdout] 245 |     pub fn error(&self) -> &ParseError<'_> {
[INFO] [stdout]     |                                       ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/parse_error.rs:288:18
[INFO] [stdout]     |
[INFO] [stdout] 288 |     pub fn error(&self) -> &Option<ParseError> {
[INFO] [stdout]     |                  ^^^^^     ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |                  |         |       |
[INFO] [stdout]     |                  |         |       the same lifetime is hidden here
[INFO] [stdout]     |                  |         the same lifetime is elided here
[INFO] [stdout]     |                  the lifetime is elided here
[INFO] [stdout]     |
[INFO] [stdout]     = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]     |
[INFO] [stdout] 288 |     pub fn error(&self) -> &Option<ParseError<'_>> {
[INFO] [stdout]     |                                              ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/parse_error.rs:341:19
[INFO] [stdout]     |
[INFO] [stdout] 341 |     pub fn errors(&self) -> &Vec<ParseError> {
[INFO] [stdout]     |                   ^^^^^     ^^^^^^^^^^^^^^^^
[INFO] [stdout]     |                   |         |    |
[INFO] [stdout]     |                   |         |    the same lifetime is hidden here
[INFO] [stdout]     |                   |         the same lifetime is elided here
[INFO] [stdout]     |                   the lifetime is elided here
[INFO] [stdout]     |
[INFO] [stdout]     = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]     |
[INFO] [stdout] 341 |     pub fn errors(&self) -> &Vec<ParseError<'_>> {
[INFO] [stdout]     |                                            ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `isohypse-tsg` (lib) due to 7 previous errors; 4 warnings emitted
[INFO] running `Command { std: "docker" "inspect" "38778ff46bbf6fddef3961c0ef9b969bb61f88925374390514258951736eef3b", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "38778ff46bbf6fddef3961c0ef9b969bb61f88925374390514258951736eef3b", kill_on_drop: false }`
[INFO] [stdout] 38778ff46bbf6fddef3961c0ef9b969bb61f88925374390514258951736eef3b
