[INFO] fetching crate kicad_parse_gen 7.0.2...
[INFO] documenting kicad_parse_gen-7.0.2 against da3f5c31e5535b3dc210067de660a45784ae1e9c for pr-162169
[INFO] extracting crate kicad_parse_gen 7.0.2 into /workspace/builds/worker-7-tc2/source
[INFO] started tweaking crates.io crate kicad_parse_gen 7.0.2
[INFO] finished tweaking crates.io crate kicad_parse_gen 7.0.2
[INFO] tweaked toml for crates.io crate kicad_parse_gen 7.0.2 written to /workspace/builds/worker-7-tc2/source/Cargo.toml
[INFO] validating manifest of crates.io crate kicad_parse_gen 7.0.2 on toolchain da3f5c31e5535b3dc210067de660a45784ae1e9c
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+da3f5c31e5535b3dc210067de660a45784ae1e9c" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }`
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+da3f5c31e5535b3dc210067de660a45784ae1e9c" "generate-lockfile" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] [stderr]     Updating crates.io index
[INFO] [stderr]      Locking 32 packages to highest compatible versions
[INFO] [stderr]       Adding difference v1.0.0 (available: v2.0.0)
[INFO] [stderr]       Adding env_logger v0.4.3 (available: v0.11.11)
[INFO] [stderr]       Adding log v0.3.9 (available: v0.4.34)
[INFO] [stderr]       Adding shellexpand v1.1.1 (available: v3.1.2)
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+da3f5c31e5535b3dc210067de660a45784ae1e9c" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] [stderr] warning: Cargo.toml: `package.edition` is unspecified, defaulting to `2015` while the latest is `2024`
[INFO] [stderr] warning: `kicad_parse_gen` (manifest) generated 1 warning
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-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:8683fc1fc2eb5c9ac98e0d076ab094b2ffac7f99da555d2b6a2e27f346de2ec7" "sleep" "infinity", kill_on_drop: false }`
[INFO] [stdout] 89d5155d9c7c1a4ceb1d8591252e23a0a7a79ec3d0471229e2f79b86257d203a
[INFO] running `Command { std: "docker" "start" "89d5155d9c7c1a4ceb1d8591252e23a0a7a79ec3d0471229e2f79b86257d203a", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "89d5155d9c7c1a4ceb1d8591252e23a0a7a79ec3d0471229e2f79b86257d203a", 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" "89d5155d9c7c1a4ceb1d8591252e23a0a7a79ec3d0471229e2f79b86257d203a" "/opt/rustwide/cargo-home/bin/cargo" "+da3f5c31e5535b3dc210067de660a45784ae1e9c" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "89d5155d9c7c1a4ceb1d8591252e23a0a7a79ec3d0471229e2f79b86257d203a", 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" "89d5155d9c7c1a4ceb1d8591252e23a0a7a79ec3d0471229e2f79b86257d203a" "/opt/rustwide/cargo-home/bin/cargo" "+da3f5c31e5535b3dc210067de660a45784ae1e9c" "doc" "--frozen" "--no-deps" "--document-private-items" "--message-format=json", kill_on_drop: false }`
[INFO] [stderr] warning: Cargo.toml: `package.edition` is unspecified, defaulting to `2015` while the latest is `2024`
[INFO] [stderr] warning: `kicad_parse_gen` (manifest) generated 1 warning
[INFO] [stderr]     Checking ucd-util v0.1.10
[INFO] [stderr]    Compiling regex v0.2.11
[INFO] [stderr]     Checking utf8-ranges v1.0.5
[INFO] [stderr]     Checking symbolic_expressions v5.0.3
[INFO] [stderr]     Checking aho-corasick v0.6.10
[INFO] [stderr]     Checking thread_local v0.3.6
[INFO] [stderr]     Checking log v0.3.9
[INFO] [stderr]     Checking dirs-sys v0.3.7
[INFO] [stderr]     Checking dirs v2.0.2
[INFO] [stderr]     Checking shellexpand v1.1.1
[INFO] [stderr]     Checking regex-syntax v0.5.6
[INFO] [stderr]     Checking env_logger v0.4.3
[INFO] [stderr]     Checking kicad_parse_gen v7.0.2 (/opt/rustwide/workdir)
[INFO] [stderr]  Documenting kicad_parse_gen v7.0.2 (/opt/rustwide/workdir)
[INFO] [stdout] warning: unexpected `cfg` condition value: `cargo-clippy`
[INFO] [stdout]    --> src/lib.rs:137:12
[INFO] [stdout]     |
[INFO] [stdout] 137 | #[cfg_attr(feature = "cargo-clippy", allow(needless_pass_by_value))]
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^^^^^^^^^^ help: remove the condition
[INFO] [stdout]     |
[INFO] [stdout]     = note: no expected values for `feature`
[INFO] [stdout]     = help: consider adding `cargo-clippy` as a feature in `Cargo.toml`
[INFO] [stdout]     = note: see <https://doc.rust-lang.org/nightly/rustc/check-cfg/cargo-specifics.html> for more information about checking conditional configuration
[INFO] [stdout]     = note: `#[warn(unexpected_cfgs)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unexpected `cfg` condition value: `cargo-clippy`
[INFO] [stdout]    --> src/symbol_lib.rs:730:12
[INFO] [stdout]     |
[INFO] [stdout] 730 | #[cfg_attr(feature = "cargo-clippy", allow(needless_pass_by_value))]
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^^^^^^^^^^ help: remove the condition
[INFO] [stdout]     |
[INFO] [stdout]     = note: no expected values for `feature`
[INFO] [stdout]     = help: consider adding `cargo-clippy` as a feature in `Cargo.toml`
[INFO] [stdout]     = note: see <https://doc.rust-lang.org/nightly/rustc/check-cfg/cargo-specifics.html> for more information about checking conditional configuration
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: trailing semicolon in macro used in expression position
[INFO] [stdout]    --> src/checkfix.rs:151:13
[INFO] [stdout]     |
[INFO] [stdout] 151 | /             info!(
[INFO] [stdout] 152 | |                 "{}{}.{} {}:{}",
[INFO] [stdout] 153 | |                 indent,
[INFO] [stdout] 154 | |                 self.section,
[INFO] [stdout] ...   |
[INFO] [stdout] 157 | |                 self.message
[INFO] [stdout] 158 | |             )
[INFO] [stdout]     | |_____________^
[INFO] [stdout]     |
[INFO] [stdout]     = note: macro invocations at the end of a block are treated as expressions
[INFO] [stdout]     = note: to ignore the value produced by the macro, add a semicolon after the invocation of `info`
[INFO] [stdout]     = warning: this was previously accepted by the compiler but is being phased out; it will become a hard error in a future release!
[INFO] [stdout]     = note: for more information, see issue #79813 <https://github.com/rust-lang/rust/issues/79813>
[INFO] [stdout]     = note: `#[warn(semicolon_in_expressions_from_non_local_macros)]` (part of `#[warn(future_incompatible)]`) on by default
[INFO] [stdout]     = note: this warning originates in the macro `info` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: trailing semicolon in macro used in expression position
[INFO] [stdout]    --> src/checkfix.rs:160:13
[INFO] [stdout]     |
[INFO] [stdout] 160 | /             warn!(
[INFO] [stdout] 161 | |                 "{}{}.{} {}:{}",
[INFO] [stdout] 162 | |                 indent,
[INFO] [stdout] 163 | |                 self.section,
[INFO] [stdout] ...   |
[INFO] [stdout] 166 | |                 self.message
[INFO] [stdout] 167 | |             )
[INFO] [stdout]     | |_____________^
[INFO] [stdout]     |
[INFO] [stdout]     = note: macro invocations at the end of a block are treated as expressions
[INFO] [stdout]     = note: to ignore the value produced by the macro, add a semicolon after the invocation of `warn`
[INFO] [stdout]     = warning: this was previously accepted by the compiler but is being phased out; it will become a hard error in a future release!
[INFO] [stdout]     = note: for more information, see issue #79813 <https://github.com/rust-lang/rust/issues/79813>
[INFO] [stdout]     = note: this warning originates in the macro `warn` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated constant `std::f64::EPSILON`: replaced by the `EPSILON` associated constant on `f64`
[INFO] [stdout]  --> src/footprint/data.rs:3:15
[INFO] [stdout]   |
[INFO] [stdout] 3 | use std::f64::EPSILON;
[INFO] [stdout]   |               ^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(deprecated)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated constant `std::f64::EPSILON`: replaced by the `EPSILON` associated constant on `f64`
[INFO] [stdout]     --> src/footprint/data.rs:1419:88
[INFO] [stdout]      |
[INFO] [stdout] 1419 |             if (reference.effects.font.size.x - reference.effects.font.size.y).abs() > EPSILON {
[INFO] [stdout]      |                                                                                        ^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated constant `std::f64::EPSILON`: replaced by the `EPSILON` associated constant on `f64`
[INFO] [stdout]     --> src/footprint/data.rs:1429:68
[INFO] [stdout]      |
[INFO] [stdout] 1429 |             if (reference.effects.font.size.y - font_size).abs() > EPSILON {
[INFO] [stdout]      |                                                                    ^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated constant `std::f64::EPSILON`: replaced by the `EPSILON` associated constant on `f64`
[INFO] [stdout]     --> src/footprint/data.rs:1439:68
[INFO] [stdout]      |
[INFO] [stdout] 1439 |             if (reference.effects.font.size.x - font_size).abs() > EPSILON {
[INFO] [stdout]      |                                                                    ^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated constant `std::f64::EPSILON`: replaced by the `EPSILON` associated constant on `f64`
[INFO] [stdout]     --> src/footprint/data.rs:1448:76
[INFO] [stdout]      |
[INFO] [stdout] 1448 |             if (reference.effects.font.thickness - font_thickness).abs() > EPSILON {
[INFO] [stdout]      |                                                                            ^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated constant `std::f64::EPSILON`: replaced by the `EPSILON` associated constant on `f64`
[INFO] [stdout]     --> src/footprint/data.rs:1493:64
[INFO] [stdout]      |
[INFO] [stdout] 1493 |             if (value.effects.font.size.y - font_size).abs() > EPSILON {
[INFO] [stdout]      |                                                                ^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated constant `std::f64::EPSILON`: replaced by the `EPSILON` associated constant on `f64`
[INFO] [stdout]     --> src/footprint/data.rs:1503:64
[INFO] [stdout]      |
[INFO] [stdout] 1503 |             if (value.effects.font.size.x - font_size).abs() > EPSILON {
[INFO] [stdout]      |                                                                ^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated constant `std::f64::EPSILON`: replaced by the `EPSILON` associated constant on `f64`
[INFO] [stdout]     --> src/footprint/data.rs:1512:72
[INFO] [stdout]      |
[INFO] [stdout] 1512 |             if (value.effects.font.thickness - font_thickness).abs() > EPSILON {
[INFO] [stdout]      |                                                                        ^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated constant `std::f64::EPSILON`: replaced by the `EPSILON` associated constant on `f64`
[INFO] [stdout]     --> src/footprint/data.rs:1546:88
[INFO] [stdout]      |
[INFO] [stdout] 1546 |             if (reference.effects.font.size.x - reference.effects.font.size.y).abs() > EPSILON {
[INFO] [stdout]      |                                                                                        ^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated constant `std::f64::EPSILON`: replaced by the `EPSILON` associated constant on `f64`
[INFO] [stdout]     --> src/footprint/data.rs:1556:68
[INFO] [stdout]      |
[INFO] [stdout] 1556 |             if (reference.effects.font.size.y - font_size).abs() > EPSILON {
[INFO] [stdout]      |                                                                    ^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated constant `std::f64::EPSILON`: replaced by the `EPSILON` associated constant on `f64`
[INFO] [stdout]     --> src/footprint/data.rs:1566:68
[INFO] [stdout]      |
[INFO] [stdout] 1566 |             if (reference.effects.font.size.x - font_size).abs() > EPSILON {
[INFO] [stdout]      |                                                                    ^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated constant `std::f64::EPSILON`: replaced by the `EPSILON` associated constant on `f64`
[INFO] [stdout]     --> src/footprint/data.rs:1575:76
[INFO] [stdout]      |
[INFO] [stdout] 1575 |             if (reference.effects.font.thickness - font_thickness).abs() > EPSILON {
[INFO] [stdout]      |                                                                            ^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated constant `std::f64::EPSILON`: replaced by the `EPSILON` associated constant on `f64`
[INFO] [stdout]    --> src/footprint/ser.rs:145:73
[INFO] [stdout]     |
[INFO] [stdout] 145 |         if self.height > 0.0 && (self.height - self.width).abs() > f64::EPSILON {
[INFO] [stdout]     |                                                                         ^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unexpected `cfg` condition value: `cargo-clippy`
[INFO] [stdout]    --> src/lib.rs:137:12
[INFO] [stdout]     |
[INFO] [stdout] 137 | #[cfg_attr(feature = "cargo-clippy", allow(needless_pass_by_value))]
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^^^^^^^^^^ help: remove the condition
[INFO] [stdout]     |
[INFO] [stdout]     = note: no expected values for `feature`
[INFO] [stdout]     = help: consider adding `cargo-clippy` as a feature in `Cargo.toml`
[INFO] [stdout]     = note: see <https://doc.rust-lang.org/nightly/rustc/check-cfg/cargo-specifics.html> for more information about checking conditional configuration
[INFO] [stdout]     = note: `#[warn(unexpected_cfgs)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unexpected `cfg` condition value: `cargo-clippy`
[INFO] [stdout]    --> src/symbol_lib.rs:730:12
[INFO] [stdout]     |
[INFO] [stdout] 730 | #[cfg_attr(feature = "cargo-clippy", allow(needless_pass_by_value))]
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^^^^^^^^^^ help: remove the condition
[INFO] [stdout]     |
[INFO] [stdout]     = note: no expected values for `feature`
[INFO] [stdout]     = help: consider adding `cargo-clippy` as a feature in `Cargo.toml`
[INFO] [stdout]     = note: see <https://doc.rust-lang.org/nightly/rustc/check-cfg/cargo-specifics.html> for more information about checking conditional configuration
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: markdown element starts on one item and ends on another
[INFO] [stdout]   --> src/util.rs:11:1
[INFO] [stdout]    |
[INFO] [stdout] 11 | / pub fn read_file<P>(path: P) -> Result<String, io::Error>
[INFO] [stdout] 12 | | where
[INFO] [stdout] 13 | |     P: AsRef<Path>,
[INFO] [stdout] ...  |
[INFO] [stdout] 18 | |     Ok(s)
[INFO] [stdout] 19 | | }
[INFO] [stdout]    | |_^
[INFO] [stdout]    |
[INFO] [stdout]    = help: the way this is parsed might change in the future
[INFO] [stdout]    = help: starts near `read file utility wrapper function`
[INFO] [stdout]    = help: ends near `read a file`
[INFO] [stdout]    = note: `#[deny(rustdoc::unportable_markdown)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: markdown element starts on one item and ends on another
[INFO] [stdout]   --> src/util.rs:11:1
[INFO] [stdout]    |
[INFO] [stdout] 11 | / pub fn read_file<P>(path: P) -> Result<String, io::Error>
[INFO] [stdout] 12 | | where
[INFO] [stdout] 13 | |     P: AsRef<Path>,
[INFO] [stdout] ...  |
[INFO] [stdout] 18 | |     Ok(s)
[INFO] [stdout] 19 | | }
[INFO] [stdout]    | |_^
[INFO] [stdout]    |
[INFO] [stdout]    = help: the way this is parsed might change in the future
[INFO] [stdout]    = help: starts near `read a file`
[INFO] [stdout]    = help: ends near `read a file`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: markdown element starts on one item and ends on another
[INFO] [stdout]   --> src/util.rs:22:1
[INFO] [stdout]    |
[INFO] [stdout] 22 | / pub fn write_file<P>(name: P, data: &str) -> Result<(), KicadError>
[INFO] [stdout] 23 | | where
[INFO] [stdout] 24 | |     P: AsRef<Path>,
[INFO] [stdout] ...  |
[INFO] [stdout] 43 | |     Ok(())
[INFO] [stdout] 44 | | }
[INFO] [stdout]    | |_^
[INFO] [stdout]    |
[INFO] [stdout]    = help: the way this is parsed might change in the future
[INFO] [stdout]    = help: starts near `write file utility wrapper function`
[INFO] [stdout]    = help: ends near `write a file`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: markdown element starts on one item and ends on another
[INFO] [stdout]   --> src/util.rs:22:1
[INFO] [stdout]    |
[INFO] [stdout] 22 | / pub fn write_file<P>(name: P, data: &str) -> Result<(), KicadError>
[INFO] [stdout] 23 | | where
[INFO] [stdout] 24 | |     P: AsRef<Path>,
[INFO] [stdout] ...  |
[INFO] [stdout] 43 | |     Ok(())
[INFO] [stdout] 44 | | }
[INFO] [stdout]    | |_^
[INFO] [stdout]    |
[INFO] [stdout]    = help: the way this is parsed might change in the future
[INFO] [stdout]    = help: starts near `write a file`
[INFO] [stdout]    = help: ends near `write a file`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not document `kicad_parse_gen`
[INFO] [stderr] warning: build failed, waiting for other jobs to finish...
[INFO] [stdout] warning: trait `Named` is never used
[INFO] [stdout]   --> src/footprint/data.rs:34:7
[INFO] [stdout]    |
[INFO] [stdout] 34 | trait Named {
[INFO] [stdout]    |       ^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `0` is never read
[INFO] [stdout]   --> src/footprint/de.rs:13:15
[INFO] [stdout]    |
[INFO] [stdout] 13 |     Thickness(f64),
[INFO] [stdout]    |     --------- ^^^
[INFO] [stdout]    |     |
[INFO] [stdout]    |     field in this variant
[INFO] [stdout]    |
[INFO] [stdout]    = note: `Part` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stdout] help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field
[INFO] [stdout]    |
[INFO] [stdout] 13 -     Thickness(f64),
[INFO] [stdout] 13 +     Thickness(()),
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: for loop over a `&mut Option`. This is more readably written as an `if let` statement
[INFO] [stdout]    --> src/layout/data.rs:101:18
[INFO] [stdout]     |
[INFO] [stdout] 101 |         for p in &mut self.fill_segments {
[INFO] [stdout]     |                  ^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(for_loops_over_fallibles)]` on by default
[INFO] [stdout] help: to check pattern in a loop use `while let`
[INFO] [stdout]     |
[INFO] [stdout] 101 -         for p in &mut self.fill_segments {
[INFO] [stdout] 101 +         while let Some(p) = &mut self.fill_segments {
[INFO] [stdout]     |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]     |
[INFO] [stdout] 101 -         for p in &mut self.fill_segments {
[INFO] [stdout] 101 +         if let Some(p) = &mut self.fill_segments {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: for loop over a `&Option`. This is more readably written as an `if let` statement
[INFO] [stdout]    --> src/layout/data.rs:116:18
[INFO] [stdout]     |
[INFO] [stdout] 116 |         for p in &self.fill_segments {
[INFO] [stdout]     |                  ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout] help: to check pattern in a loop use `while let`
[INFO] [stdout]     |
[INFO] [stdout] 116 -         for p in &self.fill_segments {
[INFO] [stdout] 116 +         while let Some(p) = &self.fill_segments {
[INFO] [stdout]     |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]     |
[INFO] [stdout] 116 -         for p in &self.fill_segments {
[INFO] [stdout] 116 +         if let Some(p) = &self.fill_segments {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] running `Command { std: "docker" "inspect" "89d5155d9c7c1a4ceb1d8591252e23a0a7a79ec3d0471229e2f79b86257d203a", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "89d5155d9c7c1a4ceb1d8591252e23a0a7a79ec3d0471229e2f79b86257d203a", kill_on_drop: false }`
[INFO] [stdout] 89d5155d9c7c1a4ceb1d8591252e23a0a7a79ec3d0471229e2f79b86257d203a
