[INFO] fetching crate gavle 0.1.1...
[INFO] testing gavle-0.1.1 against master#0d162b25edd5bf0dba9a22e83b614f1113e90474 for pr-150681
[INFO] extracting crate gavle 0.1.1 into /workspace/builds/worker-1-tc1/source
[INFO] started tweaking crates.io crate gavle 0.1.1
[INFO] finished tweaking crates.io crate gavle 0.1.1
[INFO] tweaked toml for crates.io crate gavle 0.1.1 written to /workspace/builds/worker-1-tc1/source/Cargo.toml
[INFO] validating manifest of crates.io crate gavle 0.1.1 on toolchain 0d162b25edd5bf0dba9a22e83b614f1113e90474
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+0d162b25edd5bf0dba9a22e83b614f1113e90474" "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" "+0d162b25edd5bf0dba9a22e83b614f1113e90474" "generate-lockfile" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] [stderr]     Updating crates.io index
[INFO] [stderr]      Locking 34 packages to latest compatible versions
[INFO] [stderr]       Adding arrayvec v0.6.1 (available: v0.7.6)
[INFO] [stderr]       Adding bitflags v1.3.2 (available: v2.10.0)
[INFO] [stderr]       Adding glow v0.11.2 (available: v0.16.0)
[INFO] [stderr]       Adding image v0.23.14 (available: v0.25.9)
[INFO] [stderr]       Adding thiserror v1.0.69 (available: v2.0.17)
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+0d162b25edd5bf0dba9a22e83b614f1113e90474" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] [stderr]  Downloading crates ...
[INFO] [stderr]   Downloaded arrayvec v0.6.1
[INFO] [stderr]   Downloaded glow v0.11.2
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-1-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-1-tc1/source:/opt/rustwide/workdir:ro,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" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:845e597a41426bbf2703be69acdb67d10b6de511142d05cba7bbe119c898b2c7" "/opt/rustwide/cargo-home/bin/cargo" "+0d162b25edd5bf0dba9a22e83b614f1113e90474" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] [stdout] 6f7f5ff4c6f232d286e6904c6b4f063b4ed6ceec44ff189a942685cf85077bb6
[INFO] running `Command { std: "docker" "start" "-a" "6f7f5ff4c6f232d286e6904c6b4f063b4ed6ceec44ff189a942685cf85077bb6", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "6f7f5ff4c6f232d286e6904c6b4f063b4ed6ceec44ff189a942685cf85077bb6", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "6f7f5ff4c6f232d286e6904c6b4f063b4ed6ceec44ff189a942685cf85077bb6", kill_on_drop: false }`
[INFO] [stdout] 6f7f5ff4c6f232d286e6904c6b4f063b4ed6ceec44ff189a942685cf85077bb6
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-1-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-1-tc1/source:/opt/rustwide/workdir:ro,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" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-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" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:845e597a41426bbf2703be69acdb67d10b6de511142d05cba7bbe119c898b2c7" "/opt/rustwide/cargo-home/bin/cargo" "+0d162b25edd5bf0dba9a22e83b614f1113e90474" "build" "--frozen" "--message-format=json", kill_on_drop: false }`
[INFO] [stdout] b188bcbef05dd9a52082cccb294cb5b8708bbbceeade2707e43381bba5f2a4c5
[INFO] running `Command { std: "docker" "start" "-a" "b188bcbef05dd9a52082cccb294cb5b8708bbbceeade2707e43381bba5f2a4c5", kill_on_drop: false }`
[INFO] [stderr]    Compiling tinyvec v1.10.0
[INFO] [stderr]    Compiling glow v0.11.2
[INFO] [stderr]    Compiling arrayvec v0.6.1
[INFO] [stderr]    Compiling generational-arena v0.2.9
[INFO] [stderr]    Compiling gavle v0.1.1 (/opt/rustwide/workdir)
[INFO] [stdout] warning: method `write_locks` is never used
[INFO] [stdout]   --> src/access.rs:35:5
[INFO] [stdout]    |
[INFO] [stdout] 33 | pub trait AccessLock {
[INFO] [stdout]    |           ---------- method in this trait
[INFO] [stdout] 34 |     /** The current number of active read locks on this object. */
[INFO] [stdout] 35 |     fn write_locks(&self) -> usize;
[INFO] [stdout]    |        ^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[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/lib.rs:402:19
[INFO] [stdout]     |
[INFO] [stdout] 402 |             for texture in attachments {
[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] 402 -             for texture in attachments {
[INFO] [stdout] 402 +             while let Some(texture) = attachments {
[INFO] [stdout]     |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]     |
[INFO] [stdout] 402 -             for texture in attachments {
[INFO] [stdout] 402 +             if let Some(texture) = attachments {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: call to `.clone()` on a reference in this situation does nothing
[INFO] [stdout]    --> src/lib.rs:954:48
[INFO] [stdout]     |
[INFO] [stdout] 954 |             let vertex_shader = descriptor.vertex.shader.clone();
[INFO] [stdout]     |                                                         ^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: the type `shader::VertexShader` does not implement `Clone`, so calling `clone` on `&shader::VertexShader` copies the reference, which does not do anything and can be removed
[INFO] [stdout]     = note: `#[warn(noop_method_call)]` on by default
[INFO] [stdout] help: remove this redundant call
[INFO] [stdout]     |
[INFO] [stdout] 954 -             let vertex_shader = descriptor.vertex.shader.clone();
[INFO] [stdout] 954 +             let vertex_shader = descriptor.vertex.shader;
[INFO] [stdout]     |
[INFO] [stdout] help: if you meant to clone `shader::VertexShader`, implement `Clone` for it
[INFO] [stdout]    --> src/shader.rs:61:1
[INFO] [stdout]     |
[INFO] [stdout]  61 ~         #[derive(Clone)]
[INFO] [stdout]  62 |         pub struct $name {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: call to `.clone()` on a reference in this situation does nothing
[INFO] [stdout]    --> src/lib.rs:959:49
[INFO] [stdout]     |
[INFO] [stdout] 959 |                     let fragment_shader = fragment_state.shader.clone();
[INFO] [stdout]     |                                                                ^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: the type `shader::FragmentShader` does not implement `Clone`, so calling `clone` on `&shader::FragmentShader` copies the reference, which does not do anything and can be removed
[INFO] [stdout] help: remove this redundant call
[INFO] [stdout]     |
[INFO] [stdout] 959 -                     let fragment_shader = fragment_state.shader.clone();
[INFO] [stdout] 959 +                     let fragment_shader = fragment_state.shader;
[INFO] [stdout]     |
[INFO] [stdout] help: if you meant to clone `shader::FragmentShader`, implement `Clone` for it
[INFO] [stdout]    --> src/shader.rs:61:1
[INFO] [stdout]     |
[INFO] [stdout]  61 ~         #[derive(Clone)]
[INFO] [stdout]  62 |         pub struct $name {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]   --> src/buffer.rs:84:14
[INFO] [stdout]    |
[INFO] [stdout] 84 |     fn slice<R>(&self, range: R) -> BufferSlice
[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] 84 |     fn slice<R>(&self, range: R) -> BufferSlice<'_>
[INFO] [stdout]    |                                                ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/buffer.rs:117:16
[INFO] [stdout]     |
[INFO] [stdout] 117 |               fn slice<R>(&self, range: R) -> BufferSlice
[INFO] [stdout]     |                           ^^^^^               ^^^^^^^^^^^ the same lifetime is hidden here
[INFO] [stdout]     |                           |
[INFO] [stdout]     |                           the lifetime is elided here
[INFO] [stdout] ...
[INFO] [stdout] 195 | / instance_buffers! {
[INFO] [stdout] 196 | |     #[derive(Debug)]
[INFO] [stdout] 197 | |     #[doc = "A buffer type that may be used for vertex storage."]
[INFO] [stdout] 198 | |     pub struct VertexBuffer: glow::ARRAY_BUFFER;
[INFO] [stdout] ...   |
[INFO] [stdout] 204 | |     pub struct UniformBuffer: glow::UNIFORM_BUFFER;
[INFO] [stdout] 205 | | }
[INFO] [stdout]     | |_- in this macro invocation
[INFO] [stdout]     |
[INFO] [stdout]     = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]     = note: this warning originates in the macro `instance_buffers` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]     |
[INFO] [stdout] 117 |             fn slice<R>(&self, range: R) -> BufferSlice<'_>
[INFO] [stdout]     |                                                        ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/buffer.rs:269:17
[INFO] [stdout]     |
[INFO] [stdout] 269 |     pub fn try_map(&self) -> Result<BufferView, BufferRemap> {
[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] help: use `'_` for type paths
[INFO] [stdout]     |
[INFO] [stdout] 269 |     pub fn try_map(&self) -> Result<BufferView<'_>, BufferRemap> {
[INFO] [stdout]     |                                               ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/buffer.rs:350:3
[INFO] [stdout]     |
[INFO] [stdout] 350 |         &self,
[INFO] [stdout]     |         ^^^^^ the lifetime is elided here
[INFO] [stdout] 351 |         op: BufferLoadOp) -> Result<BufferViewMut, BufferRemap> {
[INFO] [stdout]     |                                     ^^^^^^^^^^^^^ the same lifetime is hidden 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] 351 |         op: BufferLoadOp) -> Result<BufferViewMut<'_>, BufferRemap> {
[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/pipeline.rs:254:19
[INFO] [stdout]     |
[INFO] [stdout] 254 |             for texture in &fb.depth_stencil { texture.acquire_write() }
[INFO] [stdout]     |                            ^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout] help: to check pattern in a loop use `while let`
[INFO] [stdout]     |
[INFO] [stdout] 254 -             for texture in &fb.depth_stencil { texture.acquire_write() }
[INFO] [stdout] 254 +             while let Some(texture) = &fb.depth_stencil { texture.acquire_write() }
[INFO] [stdout]     |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]     |
[INFO] [stdout] 254 -             for texture in &fb.depth_stencil { texture.acquire_write() }
[INFO] [stdout] 254 +             if let Some(texture) = &fb.depth_stencil { texture.acquire_write() }
[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/pipeline.rs:256:19
[INFO] [stdout]     |
[INFO] [stdout] 256 |             for texture in &fb.depth_stencil {
[INFO] [stdout]     |                            ^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout] help: to check pattern in a loop use `while let`
[INFO] [stdout]     |
[INFO] [stdout] 256 -             for texture in &fb.depth_stencil {
[INFO] [stdout] 256 +             while let Some(texture) = &fb.depth_stencil {
[INFO] [stdout]     |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]     |
[INFO] [stdout] 256 -             for texture in &fb.depth_stencil {
[INFO] [stdout] 256 +             if let Some(texture) = &fb.depth_stencil {
[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/pipeline.rs:292:19
[INFO] [stdout]     |
[INFO] [stdout] 292 |             for texture in &fb.depth_stencil { texture.release_write() }
[INFO] [stdout]     |                            ^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout] help: to check pattern in a loop use `while let`
[INFO] [stdout]     |
[INFO] [stdout] 292 -             for texture in &fb.depth_stencil { texture.release_write() }
[INFO] [stdout] 292 +             while let Some(texture) = &fb.depth_stencil { texture.release_write() }
[INFO] [stdout]     |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]     |
[INFO] [stdout] 292 -             for texture in &fb.depth_stencil { texture.release_write() }
[INFO] [stdout] 292 +             if let Some(texture) = &fb.depth_stencil { texture.release_write() }
[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/pipeline.rs:294:19
[INFO] [stdout]     |
[INFO] [stdout] 294 |             for texture in &fb.depth_stencil { texture.release_read() }
[INFO] [stdout]     |                            ^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout] help: to check pattern in a loop use `while let`
[INFO] [stdout]     |
[INFO] [stdout] 294 -             for texture in &fb.depth_stencil { texture.release_read() }
[INFO] [stdout] 294 +             while let Some(texture) = &fb.depth_stencil { texture.release_read() }
[INFO] [stdout]     |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]     |
[INFO] [stdout] 294 -             for texture in &fb.depth_stencil { texture.release_read() }
[INFO] [stdout] 294 +             if let Some(texture) = &fb.depth_stencil { texture.release_read() }
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]   --> src/access.rs:84:27
[INFO] [stdout]    |
[INFO] [stdout] 84 |     fn acquire_write_guarded(&self) -> WriteGuard
[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] help: use `'_` for type paths
[INFO] [stdout]    |
[INFO] [stdout] 84 |     fn acquire_write_guarded(&self) -> WriteGuard<'_>
[INFO] [stdout]    |                                                  ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]   --> src/access.rs:93:26
[INFO] [stdout]    |
[INFO] [stdout] 93 |     fn acquire_read_guarded(&self) -> ReadGuard
[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] help: use `'_` for type paths
[INFO] [stdout]    |
[INFO] [stdout] 93 |     fn acquire_read_guarded(&self) -> ReadGuard<'_>
[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/framebuffer.rs:31:18
[INFO] [stdout]    |
[INFO] [stdout] 31 |         for texture in &self.depth_stencil     { assert_eq!(texture.write_locks(), a) }
[INFO] [stdout]    |                        ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout] help: to check pattern in a loop use `while let`
[INFO] [stdout]    |
[INFO] [stdout] 31 -         for texture in &self.depth_stencil     { assert_eq!(texture.write_locks(), a) }
[INFO] [stdout] 31 +         while let Some(texture) = &self.depth_stencil     { assert_eq!(texture.write_locks(), a) }
[INFO] [stdout]    |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]    |
[INFO] [stdout] 31 -         for texture in &self.depth_stencil     { assert_eq!(texture.write_locks(), a) }
[INFO] [stdout] 31 +         if let Some(texture) = &self.depth_stencil     { assert_eq!(texture.write_locks(), a) }
[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/framebuffer.rs:38:18
[INFO] [stdout]    |
[INFO] [stdout] 38 |         for texture in &self.depth_stencil     { assert_eq!(texture.read_locks(), a) }
[INFO] [stdout]    |                        ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout] help: to check pattern in a loop use `while let`
[INFO] [stdout]    |
[INFO] [stdout] 38 -         for texture in &self.depth_stencil     { assert_eq!(texture.read_locks(), a) }
[INFO] [stdout] 38 +         while let Some(texture) = &self.depth_stencil     { assert_eq!(texture.read_locks(), a) }
[INFO] [stdout]    |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]    |
[INFO] [stdout] 38 -         for texture in &self.depth_stencil     { assert_eq!(texture.read_locks(), a) }
[INFO] [stdout] 38 +         if let Some(texture) = &self.depth_stencil     { assert_eq!(texture.read_locks(), a) }
[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/framebuffer.rs:45:18
[INFO] [stdout]    |
[INFO] [stdout] 45 |         for texture in &self.depth_stencil     { texture.acquire_write(); }
[INFO] [stdout]    |                        ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout] help: to check pattern in a loop use `while let`
[INFO] [stdout]    |
[INFO] [stdout] 45 -         for texture in &self.depth_stencil     { texture.acquire_write(); }
[INFO] [stdout] 45 +         while let Some(texture) = &self.depth_stencil     { texture.acquire_write(); }
[INFO] [stdout]    |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]    |
[INFO] [stdout] 45 -         for texture in &self.depth_stencil     { texture.acquire_write(); }
[INFO] [stdout] 45 +         if let Some(texture) = &self.depth_stencil     { texture.acquire_write(); }
[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/framebuffer.rs:50:18
[INFO] [stdout]    |
[INFO] [stdout] 50 |         for texture in &self.depth_stencil     { texture.release_write(); }
[INFO] [stdout]    |                        ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout] help: to check pattern in a loop use `while let`
[INFO] [stdout]    |
[INFO] [stdout] 50 -         for texture in &self.depth_stencil     { texture.release_write(); }
[INFO] [stdout] 50 +         while let Some(texture) = &self.depth_stencil     { texture.release_write(); }
[INFO] [stdout]    |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]    |
[INFO] [stdout] 50 -         for texture in &self.depth_stencil     { texture.release_write(); }
[INFO] [stdout] 50 +         if let Some(texture) = &self.depth_stencil     { texture.release_write(); }
[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/framebuffer.rs:55:18
[INFO] [stdout]    |
[INFO] [stdout] 55 |         for texture in &self.depth_stencil     { texture.acquire_read(); }
[INFO] [stdout]    |                        ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout] help: to check pattern in a loop use `while let`
[INFO] [stdout]    |
[INFO] [stdout] 55 -         for texture in &self.depth_stencil     { texture.acquire_read(); }
[INFO] [stdout] 55 +         while let Some(texture) = &self.depth_stencil     { texture.acquire_read(); }
[INFO] [stdout]    |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]    |
[INFO] [stdout] 55 -         for texture in &self.depth_stencil     { texture.acquire_read(); }
[INFO] [stdout] 55 +         if let Some(texture) = &self.depth_stencil     { texture.acquire_read(); }
[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/framebuffer.rs:60:18
[INFO] [stdout]    |
[INFO] [stdout] 60 |         for texture in &self.depth_stencil     { texture.release_read(); }
[INFO] [stdout]    |                        ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout] help: to check pattern in a loop use `while let`
[INFO] [stdout]    |
[INFO] [stdout] 60 -         for texture in &self.depth_stencil     { texture.release_read(); }
[INFO] [stdout] 60 +         while let Some(texture) = &self.depth_stencil     { texture.release_read(); }
[INFO] [stdout]    |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]    |
[INFO] [stdout] 60 -         for texture in &self.depth_stencil     { texture.release_read(); }
[INFO] [stdout] 60 +         if let Some(texture) = &self.depth_stencil     { texture.release_read(); }
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr]     Finished `dev` profile [unoptimized + debuginfo] target(s) in 2.73s
[INFO] running `Command { std: "docker" "inspect" "b188bcbef05dd9a52082cccb294cb5b8708bbbceeade2707e43381bba5f2a4c5", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "b188bcbef05dd9a52082cccb294cb5b8708bbbceeade2707e43381bba5f2a4c5", kill_on_drop: false }`
[INFO] [stdout] b188bcbef05dd9a52082cccb294cb5b8708bbbceeade2707e43381bba5f2a4c5
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-1-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-1-tc1/source:/opt/rustwide/workdir:ro,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" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-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" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:845e597a41426bbf2703be69acdb67d10b6de511142d05cba7bbe119c898b2c7" "/opt/rustwide/cargo-home/bin/cargo" "+0d162b25edd5bf0dba9a22e83b614f1113e90474" "test" "--frozen" "--no-run" "--message-format=json", kill_on_drop: false }`
[INFO] [stdout] 0679efed2cf78af1255fe471c02c37d55ec85291ce15e7b20f92822c128c067d
[INFO] running `Command { std: "docker" "start" "-a" "0679efed2cf78af1255fe471c02c37d55ec85291ce15e7b20f92822c128c067d", kill_on_drop: false }`
[INFO] [stdout] warning: method `write_locks` is never used
[INFO] [stdout]   --> src/access.rs:35:5
[INFO] [stdout]    |
[INFO] [stdout] 33 | pub trait AccessLock {
[INFO] [stdout]    |           ---------- method in this trait
[INFO] [stdout] 34 |     /** The current number of active read locks on this object. */
[INFO] [stdout] 35 |     fn write_locks(&self) -> usize;
[INFO] [stdout]    |        ^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[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/lib.rs:402:19
[INFO] [stdout]     |
[INFO] [stdout] 402 |             for texture in attachments {
[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] 402 -             for texture in attachments {
[INFO] [stdout] 402 +             while let Some(texture) = attachments {
[INFO] [stdout]     |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]     |
[INFO] [stdout] 402 -             for texture in attachments {
[INFO] [stdout] 402 +             if let Some(texture) = attachments {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: call to `.clone()` on a reference in this situation does nothing
[INFO] [stdout]    --> src/lib.rs:954:48
[INFO] [stdout]     |
[INFO] [stdout] 954 |             let vertex_shader = descriptor.vertex.shader.clone();
[INFO] [stdout]     |                                                         ^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: the type `shader::VertexShader` does not implement `Clone`, so calling `clone` on `&shader::VertexShader` copies the reference, which does not do anything and can be removed
[INFO] [stdout]     = note: `#[warn(noop_method_call)]` on by default
[INFO] [stdout] help: remove this redundant call
[INFO] [stdout]     |
[INFO] [stdout] 954 -             let vertex_shader = descriptor.vertex.shader.clone();
[INFO] [stdout] 954 +             let vertex_shader = descriptor.vertex.shader;
[INFO] [stdout]     |
[INFO] [stdout] help: if you meant to clone `shader::VertexShader`, implement `Clone` for it
[INFO] [stdout]    --> src/shader.rs:61:1
[INFO] [stdout]     |
[INFO] [stdout]  61 ~         #[derive(Clone)]
[INFO] [stdout]  62 |         pub struct $name {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: call to `.clone()` on a reference in this situation does nothing
[INFO] [stdout]    --> src/lib.rs:959:49
[INFO] [stdout]     |
[INFO] [stdout] 959 |                     let fragment_shader = fragment_state.shader.clone();
[INFO] [stdout]     |                                                                ^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: the type `shader::FragmentShader` does not implement `Clone`, so calling `clone` on `&shader::FragmentShader` copies the reference, which does not do anything and can be removed
[INFO] [stdout] help: remove this redundant call
[INFO] [stdout]     |
[INFO] [stdout] 959 -                     let fragment_shader = fragment_state.shader.clone();
[INFO] [stdout] 959 +                     let fragment_shader = fragment_state.shader;
[INFO] [stdout]     |
[INFO] [stdout] help: if you meant to clone `shader::FragmentShader`, implement `Clone` for it
[INFO] [stdout]    --> src/shader.rs:61:1
[INFO] [stdout]     |
[INFO] [stdout]  61 ~         #[derive(Clone)]
[INFO] [stdout]  62 |         pub struct $name {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]   --> src/buffer.rs:84:14
[INFO] [stdout]    |
[INFO] [stdout] 84 |     fn slice<R>(&self, range: R) -> BufferSlice
[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] 84 |     fn slice<R>(&self, range: R) -> BufferSlice<'_>
[INFO] [stdout]    |                                                ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/buffer.rs:117:16
[INFO] [stdout]     |
[INFO] [stdout] 117 |               fn slice<R>(&self, range: R) -> BufferSlice
[INFO] [stdout]     |                           ^^^^^               ^^^^^^^^^^^ the same lifetime is hidden here
[INFO] [stdout]     |                           |
[INFO] [stdout]     |                           the lifetime is elided here
[INFO] [stdout] ...
[INFO] [stdout] 195 | / instance_buffers! {
[INFO] [stdout] 196 | |     #[derive(Debug)]
[INFO] [stdout] 197 | |     #[doc = "A buffer type that may be used for vertex storage."]
[INFO] [stdout] 198 | |     pub struct VertexBuffer: glow::ARRAY_BUFFER;
[INFO] [stdout] ...   |
[INFO] [stdout] 204 | |     pub struct UniformBuffer: glow::UNIFORM_BUFFER;
[INFO] [stdout] 205 | | }
[INFO] [stdout]     | |_- in this macro invocation
[INFO] [stdout]     |
[INFO] [stdout]     = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]     = note: this warning originates in the macro `instance_buffers` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]     |
[INFO] [stdout] 117 |             fn slice<R>(&self, range: R) -> BufferSlice<'_>
[INFO] [stdout]     |                                                        ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/buffer.rs:269:17
[INFO] [stdout]     |
[INFO] [stdout] 269 |     pub fn try_map(&self) -> Result<BufferView, BufferRemap> {
[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] help: use `'_` for type paths
[INFO] [stdout]     |
[INFO] [stdout] 269 |     pub fn try_map(&self) -> Result<BufferView<'_>, BufferRemap> {
[INFO] [stdout]     |                                               ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/buffer.rs:350:3
[INFO] [stdout]     |
[INFO] [stdout] 350 |         &self,
[INFO] [stdout]     |         ^^^^^ the lifetime is elided here
[INFO] [stdout] 351 |         op: BufferLoadOp) -> Result<BufferViewMut, BufferRemap> {
[INFO] [stdout]     |                                     ^^^^^^^^^^^^^ the same lifetime is hidden 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] 351 |         op: BufferLoadOp) -> Result<BufferViewMut<'_>, BufferRemap> {
[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/pipeline.rs:254:19
[INFO] [stdout]     |
[INFO] [stdout] 254 |             for texture in &fb.depth_stencil { texture.acquire_write() }
[INFO] [stdout]     |                            ^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout] help: to check pattern in a loop use `while let`
[INFO] [stdout]     |
[INFO] [stdout] 254 -             for texture in &fb.depth_stencil { texture.acquire_write() }
[INFO] [stdout] 254 +             while let Some(texture) = &fb.depth_stencil { texture.acquire_write() }
[INFO] [stdout]     |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]     |
[INFO] [stdout] 254 -             for texture in &fb.depth_stencil { texture.acquire_write() }
[INFO] [stdout] 254 +             if let Some(texture) = &fb.depth_stencil { texture.acquire_write() }
[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/pipeline.rs:256:19
[INFO] [stdout]     |
[INFO] [stdout] 256 |             for texture in &fb.depth_stencil {
[INFO] [stdout]     |                            ^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout] help: to check pattern in a loop use `while let`
[INFO] [stdout]     |
[INFO] [stdout] 256 -             for texture in &fb.depth_stencil {
[INFO] [stdout] 256 +             while let Some(texture) = &fb.depth_stencil {
[INFO] [stdout]     |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]     |
[INFO] [stdout] 256 -             for texture in &fb.depth_stencil {
[INFO] [stdout] 256 +             if let Some(texture) = &fb.depth_stencil {
[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/pipeline.rs:292:19
[INFO] [stdout]     |
[INFO] [stdout] 292 |             for texture in &fb.depth_stencil { texture.release_write() }
[INFO] [stdout]     |                            ^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout] help: to check pattern in a loop use `while let`
[INFO] [stdout]     |
[INFO] [stdout] 292 -             for texture in &fb.depth_stencil { texture.release_write() }
[INFO] [stdout] 292 +             while let Some(texture) = &fb.depth_stencil { texture.release_write() }
[INFO] [stdout]     |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]     |
[INFO] [stdout] 292 -             for texture in &fb.depth_stencil { texture.release_write() }
[INFO] [stdout] 292 +             if let Some(texture) = &fb.depth_stencil { texture.release_write() }
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr]    Compiling gavle v0.1.1 (/opt/rustwide/workdir)
[INFO] [stdout] warning: for loop over a `&Option`. This is more readably written as an `if let` statement
[INFO] [stdout]    --> src/pipeline.rs:294:19
[INFO] [stdout]     |
[INFO] [stdout] 294 |             for texture in &fb.depth_stencil { texture.release_read() }
[INFO] [stdout]     |                            ^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout] help: to check pattern in a loop use `while let`
[INFO] [stdout]     |
[INFO] [stdout] 294 -             for texture in &fb.depth_stencil { texture.release_read() }
[INFO] [stdout] 294 +             while let Some(texture) = &fb.depth_stencil { texture.release_read() }
[INFO] [stdout]     |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]     |
[INFO] [stdout] 294 -             for texture in &fb.depth_stencil { texture.release_read() }
[INFO] [stdout] 294 +             if let Some(texture) = &fb.depth_stencil { texture.release_read() }
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]   --> src/access.rs:84:27
[INFO] [stdout]    |
[INFO] [stdout] 84 |     fn acquire_write_guarded(&self) -> WriteGuard
[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] help: use `'_` for type paths
[INFO] [stdout]    |
[INFO] [stdout] 84 |     fn acquire_write_guarded(&self) -> WriteGuard<'_>
[INFO] [stdout]    |                                                  ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]   --> src/access.rs:93:26
[INFO] [stdout]    |
[INFO] [stdout] 93 |     fn acquire_read_guarded(&self) -> ReadGuard
[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] help: use `'_` for type paths
[INFO] [stdout]    |
[INFO] [stdout] 93 |     fn acquire_read_guarded(&self) -> ReadGuard<'_>
[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/framebuffer.rs:31:18
[INFO] [stdout]    |
[INFO] [stdout] 31 |         for texture in &self.depth_stencil     { assert_eq!(texture.write_locks(), a) }
[INFO] [stdout]    |                        ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout] help: to check pattern in a loop use `while let`
[INFO] [stdout]    |
[INFO] [stdout] 31 -         for texture in &self.depth_stencil     { assert_eq!(texture.write_locks(), a) }
[INFO] [stdout] 31 +         while let Some(texture) = &self.depth_stencil     { assert_eq!(texture.write_locks(), a) }
[INFO] [stdout]    |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]    |
[INFO] [stdout] 31 -         for texture in &self.depth_stencil     { assert_eq!(texture.write_locks(), a) }
[INFO] [stdout] 31 +         if let Some(texture) = &self.depth_stencil     { assert_eq!(texture.write_locks(), a) }
[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/framebuffer.rs:38:18
[INFO] [stdout]    |
[INFO] [stdout] 38 |         for texture in &self.depth_stencil     { assert_eq!(texture.read_locks(), a) }
[INFO] [stdout]    |                        ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout] help: to check pattern in a loop use `while let`
[INFO] [stdout]    |
[INFO] [stdout] 38 -         for texture in &self.depth_stencil     { assert_eq!(texture.read_locks(), a) }
[INFO] [stdout] 38 +         while let Some(texture) = &self.depth_stencil     { assert_eq!(texture.read_locks(), a) }
[INFO] [stdout]    |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]    |
[INFO] [stdout] 38 -         for texture in &self.depth_stencil     { assert_eq!(texture.read_locks(), a) }
[INFO] [stdout] 38 +         if let Some(texture) = &self.depth_stencil     { assert_eq!(texture.read_locks(), a) }
[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/framebuffer.rs:45:18
[INFO] [stdout]    |
[INFO] [stdout] 45 |         for texture in &self.depth_stencil     { texture.acquire_write(); }
[INFO] [stdout]    |                        ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout] help: to check pattern in a loop use `while let`
[INFO] [stdout]    |
[INFO] [stdout] 45 -         for texture in &self.depth_stencil     { texture.acquire_write(); }
[INFO] [stdout] 45 +         while let Some(texture) = &self.depth_stencil     { texture.acquire_write(); }
[INFO] [stdout]    |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]    |
[INFO] [stdout] 45 -         for texture in &self.depth_stencil     { texture.acquire_write(); }
[INFO] [stdout] 45 +         if let Some(texture) = &self.depth_stencil     { texture.acquire_write(); }
[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/framebuffer.rs:50:18
[INFO] [stdout]    |
[INFO] [stdout] 50 |         for texture in &self.depth_stencil     { texture.release_write(); }
[INFO] [stdout]    |                        ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout] help: to check pattern in a loop use `while let`
[INFO] [stdout]    |
[INFO] [stdout] 50 -         for texture in &self.depth_stencil     { texture.release_write(); }
[INFO] [stdout] 50 +         while let Some(texture) = &self.depth_stencil     { texture.release_write(); }
[INFO] [stdout]    |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]    |
[INFO] [stdout] 50 -         for texture in &self.depth_stencil     { texture.release_write(); }
[INFO] [stdout] 50 +         if let Some(texture) = &self.depth_stencil     { texture.release_write(); }
[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/framebuffer.rs:55:18
[INFO] [stdout]    |
[INFO] [stdout] 55 |         for texture in &self.depth_stencil     { texture.acquire_read(); }
[INFO] [stdout]    |                        ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout] help: to check pattern in a loop use `while let`
[INFO] [stdout]    |
[INFO] [stdout] 55 -         for texture in &self.depth_stencil     { texture.acquire_read(); }
[INFO] [stdout] 55 +         while let Some(texture) = &self.depth_stencil     { texture.acquire_read(); }
[INFO] [stdout]    |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]    |
[INFO] [stdout] 55 -         for texture in &self.depth_stencil     { texture.acquire_read(); }
[INFO] [stdout] 55 +         if let Some(texture) = &self.depth_stencil     { texture.acquire_read(); }
[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/framebuffer.rs:60:18
[INFO] [stdout]    |
[INFO] [stdout] 60 |         for texture in &self.depth_stencil     { texture.release_read(); }
[INFO] [stdout]    |                        ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout] help: to check pattern in a loop use `while let`
[INFO] [stdout]    |
[INFO] [stdout] 60 -         for texture in &self.depth_stencil     { texture.release_read(); }
[INFO] [stdout] 60 +         while let Some(texture) = &self.depth_stencil     { texture.release_read(); }
[INFO] [stdout]    |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]    |
[INFO] [stdout] 60 -         for texture in &self.depth_stencil     { texture.release_read(); }
[INFO] [stdout] 60 +         if let Some(texture) = &self.depth_stencil     { texture.release_read(); }
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `write_locks` is never used
[INFO] [stdout]   --> src/access.rs:35:5
[INFO] [stdout]    |
[INFO] [stdout] 33 | pub trait AccessLock {
[INFO] [stdout]    |           ---------- method in this trait
[INFO] [stdout] 34 |     /** The current number of active read locks on this object. */
[INFO] [stdout] 35 |     fn write_locks(&self) -> usize;
[INFO] [stdout]    |        ^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[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/lib.rs:402:19
[INFO] [stdout]     |
[INFO] [stdout] 402 |             for texture in attachments {
[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] 402 -             for texture in attachments {
[INFO] [stdout] 402 +             while let Some(texture) = attachments {
[INFO] [stdout]     |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]     |
[INFO] [stdout] 402 -             for texture in attachments {
[INFO] [stdout] 402 +             if let Some(texture) = attachments {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: call to `.clone()` on a reference in this situation does nothing
[INFO] [stdout]    --> src/lib.rs:954:48
[INFO] [stdout]     |
[INFO] [stdout] 954 |             let vertex_shader = descriptor.vertex.shader.clone();
[INFO] [stdout]     |                                                         ^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: the type `shader::VertexShader` does not implement `Clone`, so calling `clone` on `&shader::VertexShader` copies the reference, which does not do anything and can be removed
[INFO] [stdout]     = note: `#[warn(noop_method_call)]` on by default
[INFO] [stdout] help: remove this redundant call
[INFO] [stdout]     |
[INFO] [stdout] 954 -             let vertex_shader = descriptor.vertex.shader.clone();
[INFO] [stdout] 954 +             let vertex_shader = descriptor.vertex.shader;
[INFO] [stdout]     |
[INFO] [stdout] help: if you meant to clone `shader::VertexShader`, implement `Clone` for it
[INFO] [stdout]    --> src/shader.rs:61:1
[INFO] [stdout]     |
[INFO] [stdout]  61 ~         #[derive(Clone)]
[INFO] [stdout]  62 |         pub struct $name {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: call to `.clone()` on a reference in this situation does nothing
[INFO] [stdout]    --> src/lib.rs:959:49
[INFO] [stdout]     |
[INFO] [stdout] 959 |                     let fragment_shader = fragment_state.shader.clone();
[INFO] [stdout]     |                                                                ^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: the type `shader::FragmentShader` does not implement `Clone`, so calling `clone` on `&shader::FragmentShader` copies the reference, which does not do anything and can be removed
[INFO] [stdout] help: remove this redundant call
[INFO] [stdout]     |
[INFO] [stdout] 959 -                     let fragment_shader = fragment_state.shader.clone();
[INFO] [stdout] 959 +                     let fragment_shader = fragment_state.shader;
[INFO] [stdout]     |
[INFO] [stdout] help: if you meant to clone `shader::FragmentShader`, implement `Clone` for it
[INFO] [stdout]    --> src/shader.rs:61:1
[INFO] [stdout]     |
[INFO] [stdout]  61 ~         #[derive(Clone)]
[INFO] [stdout]  62 |         pub struct $name {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]   --> src/buffer.rs:84:14
[INFO] [stdout]    |
[INFO] [stdout] 84 |     fn slice<R>(&self, range: R) -> BufferSlice
[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] 84 |     fn slice<R>(&self, range: R) -> BufferSlice<'_>
[INFO] [stdout]    |                                                ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/buffer.rs:117:16
[INFO] [stdout]     |
[INFO] [stdout] 117 |               fn slice<R>(&self, range: R) -> BufferSlice
[INFO] [stdout]     |                           ^^^^^               ^^^^^^^^^^^ the same lifetime is hidden here
[INFO] [stdout]     |                           |
[INFO] [stdout]     |                           the lifetime is elided here
[INFO] [stdout] ...
[INFO] [stdout] 195 | / instance_buffers! {
[INFO] [stdout] 196 | |     #[derive(Debug)]
[INFO] [stdout] 197 | |     #[doc = "A buffer type that may be used for vertex storage."]
[INFO] [stdout] 198 | |     pub struct VertexBuffer: glow::ARRAY_BUFFER;
[INFO] [stdout] ...   |
[INFO] [stdout] 204 | |     pub struct UniformBuffer: glow::UNIFORM_BUFFER;
[INFO] [stdout] 205 | | }
[INFO] [stdout]     | |_- in this macro invocation
[INFO] [stdout]     |
[INFO] [stdout]     = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]     = note: this warning originates in the macro `instance_buffers` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]     |
[INFO] [stdout] 117 |             fn slice<R>(&self, range: R) -> BufferSlice<'_>
[INFO] [stdout]     |                                                        ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/buffer.rs:269:17
[INFO] [stdout]     |
[INFO] [stdout] 269 |     pub fn try_map(&self) -> Result<BufferView, BufferRemap> {
[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] help: use `'_` for type paths
[INFO] [stdout]     |
[INFO] [stdout] 269 |     pub fn try_map(&self) -> Result<BufferView<'_>, BufferRemap> {
[INFO] [stdout]     |                                               ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/buffer.rs:350:3
[INFO] [stdout]     |
[INFO] [stdout] 350 |         &self,
[INFO] [stdout]     |         ^^^^^ the lifetime is elided here
[INFO] [stdout] 351 |         op: BufferLoadOp) -> Result<BufferViewMut, BufferRemap> {
[INFO] [stdout]     |                                     ^^^^^^^^^^^^^ the same lifetime is hidden 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] 351 |         op: BufferLoadOp) -> Result<BufferViewMut<'_>, BufferRemap> {
[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/pipeline.rs:254:19
[INFO] [stdout]     |
[INFO] [stdout] 254 |             for texture in &fb.depth_stencil { texture.acquire_write() }
[INFO] [stdout]     |                            ^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout] help: to check pattern in a loop use `while let`
[INFO] [stdout]     |
[INFO] [stdout] 254 -             for texture in &fb.depth_stencil { texture.acquire_write() }
[INFO] [stdout] 254 +             while let Some(texture) = &fb.depth_stencil { texture.acquire_write() }
[INFO] [stdout]     |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]     |
[INFO] [stdout] 254 -             for texture in &fb.depth_stencil { texture.acquire_write() }
[INFO] [stdout] 254 +             if let Some(texture) = &fb.depth_stencil { texture.acquire_write() }
[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/pipeline.rs:256:19
[INFO] [stdout]     |
[INFO] [stdout] 256 |             for texture in &fb.depth_stencil {
[INFO] [stdout]     |                            ^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout] help: to check pattern in a loop use `while let`
[INFO] [stdout]     |
[INFO] [stdout] 256 -             for texture in &fb.depth_stencil {
[INFO] [stdout] 256 +             while let Some(texture) = &fb.depth_stencil {
[INFO] [stdout]     |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]     |
[INFO] [stdout] 256 -             for texture in &fb.depth_stencil {
[INFO] [stdout] 256 +             if let Some(texture) = &fb.depth_stencil {
[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/pipeline.rs:292:19
[INFO] [stdout]     |
[INFO] [stdout] 292 |             for texture in &fb.depth_stencil { texture.release_write() }
[INFO] [stdout]     |                            ^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout] help: to check pattern in a loop use `while let`
[INFO] [stdout]     |
[INFO] [stdout] 292 -             for texture in &fb.depth_stencil { texture.release_write() }
[INFO] [stdout] 292 +             while let Some(texture) = &fb.depth_stencil { texture.release_write() }
[INFO] [stdout]     |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]     |
[INFO] [stdout] 292 -             for texture in &fb.depth_stencil { texture.release_write() }
[INFO] [stdout] 292 +             if let Some(texture) = &fb.depth_stencil { texture.release_write() }
[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/pipeline.rs:294:19
[INFO] [stdout]     |
[INFO] [stdout] 294 |             for texture in &fb.depth_stencil { texture.release_read() }
[INFO] [stdout]     |                            ^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout] help: to check pattern in a loop use `while let`
[INFO] [stdout]     |
[INFO] [stdout] 294 -             for texture in &fb.depth_stencil { texture.release_read() }
[INFO] [stdout] 294 +             while let Some(texture) = &fb.depth_stencil { texture.release_read() }
[INFO] [stdout]     |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]     |
[INFO] [stdout] 294 -             for texture in &fb.depth_stencil { texture.release_read() }
[INFO] [stdout] 294 +             if let Some(texture) = &fb.depth_stencil { texture.release_read() }
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]   --> src/access.rs:84:27
[INFO] [stdout]    |
[INFO] [stdout] 84 |     fn acquire_write_guarded(&self) -> WriteGuard
[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] help: use `'_` for type paths
[INFO] [stdout]    |
[INFO] [stdout] 84 |     fn acquire_write_guarded(&self) -> WriteGuard<'_>
[INFO] [stdout]    |                                                  ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]   --> src/access.rs:93:26
[INFO] [stdout]    |
[INFO] [stdout] 93 |     fn acquire_read_guarded(&self) -> ReadGuard
[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] help: use `'_` for type paths
[INFO] [stdout]    |
[INFO] [stdout] 93 |     fn acquire_read_guarded(&self) -> ReadGuard<'_>
[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/framebuffer.rs:31:18
[INFO] [stdout]    |
[INFO] [stdout] 31 |         for texture in &self.depth_stencil     { assert_eq!(texture.write_locks(), a) }
[INFO] [stdout]    |                        ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout] help: to check pattern in a loop use `while let`
[INFO] [stdout]    |
[INFO] [stdout] 31 -         for texture in &self.depth_stencil     { assert_eq!(texture.write_locks(), a) }
[INFO] [stdout] 31 +         while let Some(texture) = &self.depth_stencil     { assert_eq!(texture.write_locks(), a) }
[INFO] [stdout]    |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]    |
[INFO] [stdout] 31 -         for texture in &self.depth_stencil     { assert_eq!(texture.write_locks(), a) }
[INFO] [stdout] 31 +         if let Some(texture) = &self.depth_stencil     { assert_eq!(texture.write_locks(), a) }
[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/framebuffer.rs:38:18
[INFO] [stdout]    |
[INFO] [stdout] 38 |         for texture in &self.depth_stencil     { assert_eq!(texture.read_locks(), a) }
[INFO] [stdout]    |                        ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout] help: to check pattern in a loop use `while let`
[INFO] [stdout]    |
[INFO] [stdout] 38 -         for texture in &self.depth_stencil     { assert_eq!(texture.read_locks(), a) }
[INFO] [stdout] 38 +         while let Some(texture) = &self.depth_stencil     { assert_eq!(texture.read_locks(), a) }
[INFO] [stdout]    |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]    |
[INFO] [stdout] 38 -         for texture in &self.depth_stencil     { assert_eq!(texture.read_locks(), a) }
[INFO] [stdout] 38 +         if let Some(texture) = &self.depth_stencil     { assert_eq!(texture.read_locks(), a) }
[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/framebuffer.rs:45:18
[INFO] [stdout]    |
[INFO] [stdout] 45 |         for texture in &self.depth_stencil     { texture.acquire_write(); }
[INFO] [stdout]    |                        ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout] help: to check pattern in a loop use `while let`
[INFO] [stdout]    |
[INFO] [stdout] 45 -         for texture in &self.depth_stencil     { texture.acquire_write(); }
[INFO] [stdout] 45 +         while let Some(texture) = &self.depth_stencil     { texture.acquire_write(); }
[INFO] [stdout]    |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]    |
[INFO] [stdout] 45 -         for texture in &self.depth_stencil     { texture.acquire_write(); }
[INFO] [stdout] 45 +         if let Some(texture) = &self.depth_stencil     { texture.acquire_write(); }
[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/framebuffer.rs:50:18
[INFO] [stdout]    |
[INFO] [stdout] 50 |         for texture in &self.depth_stencil     { texture.release_write(); }
[INFO] [stdout]    |                        ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout] help: to check pattern in a loop use `while let`
[INFO] [stdout]    |
[INFO] [stdout] 50 -         for texture in &self.depth_stencil     { texture.release_write(); }
[INFO] [stdout] 50 +         while let Some(texture) = &self.depth_stencil     { texture.release_write(); }
[INFO] [stdout]    |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]    |
[INFO] [stdout] 50 -         for texture in &self.depth_stencil     { texture.release_write(); }
[INFO] [stdout] 50 +         if let Some(texture) = &self.depth_stencil     { texture.release_write(); }
[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/framebuffer.rs:55:18
[INFO] [stdout]    |
[INFO] [stdout] 55 |         for texture in &self.depth_stencil     { texture.acquire_read(); }
[INFO] [stdout]    |                        ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout] help: to check pattern in a loop use `while let`
[INFO] [stdout]    |
[INFO] [stdout] 55 -         for texture in &self.depth_stencil     { texture.acquire_read(); }
[INFO] [stdout] 55 +         while let Some(texture) = &self.depth_stencil     { texture.acquire_read(); }
[INFO] [stdout]    |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]    |
[INFO] [stdout] 55 -         for texture in &self.depth_stencil     { texture.acquire_read(); }
[INFO] [stdout] 55 +         if let Some(texture) = &self.depth_stencil     { texture.acquire_read(); }
[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/framebuffer.rs:60:18
[INFO] [stdout]    |
[INFO] [stdout] 60 |         for texture in &self.depth_stencil     { texture.release_read(); }
[INFO] [stdout]    |                        ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout] help: to check pattern in a loop use `while let`
[INFO] [stdout]    |
[INFO] [stdout] 60 -         for texture in &self.depth_stencil     { texture.release_read(); }
[INFO] [stdout] 60 +         while let Some(texture) = &self.depth_stencil     { texture.release_read(); }
[INFO] [stdout]    |
[INFO] [stdout] help: consider using `if let` to clear intent
[INFO] [stdout]    |
[INFO] [stdout] 60 -         for texture in &self.depth_stencil     { texture.release_read(); }
[INFO] [stdout] 60 +         if let Some(texture) = &self.depth_stencil     { texture.release_read(); }
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr]     Finished `test` profile [unoptimized + debuginfo] target(s) in 1.07s
[INFO] running `Command { std: "docker" "inspect" "0679efed2cf78af1255fe471c02c37d55ec85291ce15e7b20f92822c128c067d", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "0679efed2cf78af1255fe471c02c37d55ec85291ce15e7b20f92822c128c067d", kill_on_drop: false }`
[INFO] [stdout] 0679efed2cf78af1255fe471c02c37d55ec85291ce15e7b20f92822c128c067d
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-1-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-1-tc1/source:/opt/rustwide/workdir:ro,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" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-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" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:845e597a41426bbf2703be69acdb67d10b6de511142d05cba7bbe119c898b2c7" "/opt/rustwide/cargo-home/bin/cargo" "+0d162b25edd5bf0dba9a22e83b614f1113e90474" "test" "--frozen", kill_on_drop: false }`
[INFO] [stdout] ba3b9345be999423255a08882e09ef70230a3ba18bf53eee093e36111e0225cc
[INFO] running `Command { std: "docker" "start" "-a" "ba3b9345be999423255a08882e09ef70230a3ba18bf53eee093e36111e0225cc", kill_on_drop: false }`
[INFO] [stderr] warning: method `write_locks` is never used
[INFO] [stderr]   --> src/access.rs:35:5
[INFO] [stderr]    |
[INFO] [stderr] 33 | pub trait AccessLock {
[INFO] [stderr]    |           ---------- method in this trait
[INFO] [stderr] 34 |     /** The current number of active read locks on this object. */
[INFO] [stderr] 35 |     fn write_locks(&self) -> usize;
[INFO] [stderr]    |        ^^^^^^^^^^^
[INFO] [stderr]    |
[INFO] [stderr]    = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[INFO] [stderr] 
[INFO] [stderr] warning: for loop over a `&Option`. This is more readably written as an `if let` statement
[INFO] [stderr]    --> src/lib.rs:402:19
[INFO] [stderr]     |
[INFO] [stderr] 402 |             for texture in attachments {
[INFO] [stderr]     |                            ^^^^^^^^^^^
[INFO] [stderr]     |
[INFO] [stderr]     = note: `#[warn(for_loops_over_fallibles)]` on by default
[INFO] [stderr] help: to check pattern in a loop use `while let`
[INFO] [stderr]     |
[INFO] [stderr] 402 -             for texture in attachments {
[INFO] [stderr] 402 +             while let Some(texture) = attachments {
[INFO] [stderr]     |
[INFO] [stderr] help: consider using `if let` to clear intent
[INFO] [stderr]     |
[INFO] [stderr] 402 -             for texture in attachments {
[INFO] [stderr] 402 +             if let Some(texture) = attachments {
[INFO] [stderr]     |
[INFO] [stderr] 
[INFO] [stderr] warning: call to `.clone()` on a reference in this situation does nothing
[INFO] [stderr]    --> src/lib.rs:954:48
[INFO] [stderr]     |
[INFO] [stderr] 954 |             let vertex_shader = descriptor.vertex.shader.clone();
[INFO] [stderr]     |                                                         ^^^^^^^^
[INFO] [stderr]     |
[INFO] [stderr]     = note: the type `shader::VertexShader` does not implement `Clone`, so calling `clone` on `&shader::VertexShader` copies the reference, which does not do anything and can be removed
[INFO] [stderr]     = note: `#[warn(noop_method_call)]` on by default
[INFO] [stderr] help: remove this redundant call
[INFO] [stderr]     |
[INFO] [stderr] 954 -             let vertex_shader = descriptor.vertex.shader.clone();
[INFO] [stderr] 954 +             let vertex_shader = descriptor.vertex.shader;
[INFO] [stderr]     |
[INFO] [stderr] help: if you meant to clone `shader::VertexShader`, implement `Clone` for it
[INFO] [stderr]    --> src/shader.rs:61:1
[INFO] [stderr]     |
[INFO] [stderr]  61 ~         #[derive(Clone)]
[INFO] [stderr]  62 |         pub struct $name {
[INFO] [stderr]     |
[INFO] [stderr] 
[INFO] [stderr] warning: call to `.clone()` on a reference in this situation does nothing
[INFO] [stderr]    --> src/lib.rs:959:49
[INFO] [stderr]     |
[INFO] [stderr] 959 |                     let fragment_shader = fragment_state.shader.clone();
[INFO] [stderr]     |                                                                ^^^^^^^^
[INFO] [stderr]     |
[INFO] [stderr]     = note: the type `shader::FragmentShader` does not implement `Clone`, so calling `clone` on `&shader::FragmentShader` copies the reference, which does not do anything and can be removed
[INFO] [stderr] help: remove this redundant call
[INFO] [stderr]     |
[INFO] [stderr] 959 -                     let fragment_shader = fragment_state.shader.clone();
[INFO] [stderr] 959 +                     let fragment_shader = fragment_state.shader;
[INFO] [stderr]     |
[INFO] [stderr] help: if you meant to clone `shader::FragmentShader`, implement `Clone` for it
[INFO] [stderr]    --> src/shader.rs:61:1
[INFO] [stderr]     |
[INFO] [stderr]  61 ~         #[derive(Clone)]
[INFO] [stderr]  62 |         pub struct $name {
[INFO] [stderr]     |
[INFO] [stderr] 
[INFO] [stderr] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stderr]   --> src/buffer.rs:84:14
[INFO] [stderr]    |
[INFO] [stderr] 84 |     fn slice<R>(&self, range: R) -> BufferSlice
[INFO] [stderr]    |                 ^^^^^               ^^^^^^^^^^^ the same lifetime is hidden here
[INFO] [stderr]    |                 |
[INFO] [stderr]    |                 the lifetime is elided here
[INFO] [stderr]    |
[INFO] [stderr]    = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stderr]    = note: `#[warn(mismatched_lifetime_syntaxes)]` on by default
[INFO] [stderr] help: use `'_` for type paths
[INFO] [stderr]    |
[INFO] [stderr] 84 |     fn slice<R>(&self, range: R) -> BufferSlice<'_>
[INFO] [stderr]    |                                                ++++
[INFO] [stderr] 
[INFO] [stderr] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stderr]    --> src/buffer.rs:117:16
[INFO] [stderr]     |
[INFO] [stderr] 117 |               fn slice<R>(&self, range: R) -> BufferSlice
[INFO] [stderr]     |                           ^^^^^               ^^^^^^^^^^^ the same lifetime is hidden here
[INFO] [stderr]     |                           |
[INFO] [stderr]     |                           the lifetime is elided here
[INFO] [stderr] ...
[INFO] [stderr] 195 | / instance_buffers! {
[INFO] [stderr] 196 | |     #[derive(Debug)]
[INFO] [stderr] 197 | |     #[doc = "A buffer type that may be used for vertex storage."]
[INFO] [stderr] 198 | |     pub struct VertexBuffer: glow::ARRAY_BUFFER;
[INFO] [stderr] ...   |
[INFO] [stderr] 204 | |     pub struct UniformBuffer: glow::UNIFORM_BUFFER;
[INFO] [stderr] 205 | | }
[INFO] [stderr]     | |_- in this macro invocation
[INFO] [stderr]     |
[INFO] [stderr]     = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stderr]     = note: this warning originates in the macro `instance_buffers` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stderr] help: use `'_` for type paths
[INFO] [stderr]     |
[INFO] [stderr] 117 |             fn slice<R>(&self, range: R) -> BufferSlice<'_>
[INFO] [stderr]     |                                                        ++++
[INFO] [stderr] 
[INFO] [stderr] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stderr]    --> src/buffer.rs:269:17
[INFO] [stderr]     |
[INFO] [stderr] 269 |     pub fn try_map(&self) -> Result<BufferView, BufferRemap> {
[INFO] [stderr]     |                    ^^^^^            ^^^^^^^^^^ the same lifetime is hidden here
[INFO] [stderr]     |                    |
[INFO] [stderr]     |                    the lifetime is elided here
[INFO] [stderr]     |
[INFO] [stderr]     = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stderr] help: use `'_` for type paths
[INFO] [stderr]     |
[INFO] [stderr] 269 |     pub fn try_map(&self) -> Result<BufferView<'_>, BufferRemap> {
[INFO] [stderr]     |                                               ++++
[INFO] [stderr] 
[INFO] [stderr] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stderr]    --> src/buffer.rs:350:3
[INFO] [stderr]     |
[INFO] [stderr] 350 |         &self,
[INFO] [stderr]     |         ^^^^^ the lifetime is elided here
[INFO] [stderr] 351 |         op: BufferLoadOp) -> Result<BufferViewMut, BufferRemap> {
[INFO] [stderr]     |                                     ^^^^^^^^^^^^^ the same lifetime is hidden here
[INFO] [stderr]     |
[INFO] [stderr]     = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stderr] help: use `'_` for type paths
[INFO] [stderr]     |
[INFO] [stderr] 351 |         op: BufferLoadOp) -> Result<BufferViewMut<'_>, BufferRemap> {
[INFO] [stderr]     |                                                  ++++
[INFO] [stderr] 
[INFO] [stderr] warning: for loop over a `&Option`. This is more readably written as an `if let` statement
[INFO] [stderr]    --> src/pipeline.rs:254:19
[INFO] [stderr]     |
[INFO] [stderr] 254 |             for texture in &fb.depth_stencil { texture.acquire_write() }
[INFO] [stderr]     |                            ^^^^^^^^^^^^^^^^^
[INFO] [stderr]     |
[INFO] [stderr] help: to check pattern in a loop use `while let`
[INFO] [stderr]     |
[INFO] [stderr] 254 -             for texture in &fb.depth_stencil { texture.acquire_write() }
[INFO] [stderr] 254 +             while let Some(texture) = &fb.depth_stencil { texture.acquire_write() }
[INFO] [stderr]     |
[INFO] [stderr] help: consider using `if let` to clear intent
[INFO] [stderr]     |
[INFO] [stderr] 254 -             for texture in &fb.depth_stencil { texture.acquire_write() }
[INFO] [stderr] 254 +             if let Some(texture) = &fb.depth_stencil { texture.acquire_write() }
[INFO] [stderr]     |
[INFO] [stderr] 
[INFO] [stderr] warning: for loop over a `&Option`. This is more readably written as an `if let` statement
[INFO] [stderr]    --> src/pipeline.rs:256:19
[INFO] [stderr]     |
[INFO] [stderr] 256 |             for texture in &fb.depth_stencil {
[INFO] [stderr]     |                            ^^^^^^^^^^^^^^^^^
[INFO] [stderr]     |
[INFO] [stderr] help: to check pattern in a loop use `while let`
[INFO] [stderr]     |
[INFO] [stderr] 256 -             for texture in &fb.depth_stencil {
[INFO] [stderr] 256 +             while let Some(texture) = &fb.depth_stencil {
[INFO] [stderr]     |
[INFO] [stderr] help: consider using `if let` to clear intent
[INFO] [stderr]     |
[INFO] [stderr] 256 -             for texture in &fb.depth_stencil {
[INFO] [stderr] 256 +             if let Some(texture) = &fb.depth_stencil {
[INFO] [stderr]     |
[INFO] [stderr] 
[INFO] [stderr] warning: for loop over a `&Option`. This is more readably written as an `if let` statement
[INFO] [stderr]    --> src/pipeline.rs:292:19
[INFO] [stderr]     |
[INFO] [stderr] 292 |             for texture in &fb.depth_stencil { texture.release_write() }
[INFO] [stderr]     |                            ^^^^^^^^^^^^^^^^^
[INFO] [stderr]     |
[INFO] [stderr] help: to check pattern in a loop use `while let`
[INFO] [stderr]     |
[INFO] [stderr] 292 -             for texture in &fb.depth_stencil { texture.release_write() }
[INFO] [stderr] 292 +             while let Some(texture) = &fb.depth_stencil { texture.release_write() }
[INFO] [stderr]     |
[INFO] [stderr] help: consider using `if let` to clear intent
[INFO] [stderr]     |
[INFO] [stderr] 292 -             for texture in &fb.depth_stencil { texture.release_write() }
[INFO] [stderr] 292 +             if let Some(texture) = &fb.depth_stencil { texture.release_write() }
[INFO] [stderr]     |
[INFO] [stderr] 
[INFO] [stderr] warning: for loop over a `&Option`. This is more readably written as an `if let` statement
[INFO] [stderr]    --> src/pipeline.rs:294:19
[INFO] [stderr]     |
[INFO] [stderr] 294 |             for texture in &fb.depth_stencil { texture.release_read() }
[INFO] [stderr]     |                            ^^^^^^^^^^^^^^^^^
[INFO] [stderr]     |
[INFO] [stderr] help: to check pattern in a loop use `while let`
[INFO] [stderr]     |
[INFO] [stderr] 294 -             for texture in &fb.depth_stencil { texture.release_read() }
[INFO] [stderr] 294 +             while let Some(texture) = &fb.depth_stencil { texture.release_read() }
[INFO] [stderr]     |
[INFO] [stderr] help: consider using `if let` to clear intent
[INFO] [stderr]     |
[INFO] [stderr] 294 -             for texture in &fb.depth_stencil { texture.release_read() }
[INFO] [stderr] 294 +             if let Some(texture) = &fb.depth_stencil { texture.release_read() }
[INFO] [stderr]     |
[INFO] [stderr] 
[INFO] [stderr] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stderr]   --> src/access.rs:84:27
[INFO] [stderr]    |
[INFO] [stderr] 84 |     fn acquire_write_guarded(&self) -> WriteGuard
[INFO] [stderr]    |                              ^^^^^     ^^^^^^^^^^ the same lifetime is hidden here
[INFO] [stderr]    |                              |
[INFO] [stderr]    |                              the lifetime is elided here
[INFO] [stderr]    |
[INFO] [stderr]    = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stderr] help: use `'_` for type paths
[INFO] [stderr]    |
[INFO] [stderr] 84 |     fn acquire_write_guarded(&self) -> WriteGuard<'_>
[INFO] [stderr]    |                                                  ++++
[INFO] [stderr] 
[INFO] [stderr] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stderr]   --> src/access.rs:93:26
[INFO] [stderr]    |
[INFO] [stderr] 93 |     fn acquire_read_guarded(&self) -> ReadGuard
[INFO] [stderr]    |                             ^^^^^     ^^^^^^^^^ the same lifetime is hidden here
[INFO] [stderr]    |                             |
[INFO] [stderr]    |                             the lifetime is elided here
[INFO] [stderr]    |
[INFO] [stderr]    = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stderr] help: use `'_` for type paths
[INFO] [stderr]    |
[INFO] [stderr] 93 |     fn acquire_read_guarded(&self) -> ReadGuard<'_>
[INFO] [stderr]    |                                                ++++
[INFO] [stderr] 
[INFO] [stderr] warning: for loop over a `&Option`. This is more readably written as an `if let` statement
[INFO] [stderr]   --> src/framebuffer.rs:31:18
[INFO] [stderr]    |
[INFO] [stderr] 31 |         for texture in &self.depth_stencil     { assert_eq!(texture.write_locks(), a) }
[INFO] [stderr]    |                        ^^^^^^^^^^^^^^^^^^^
[INFO] [stderr]    |
[INFO] [stderr] help: to check pattern in a loop use `while let`
[INFO] [stderr]    |
[INFO] [stderr] 31 -         for texture in &self.depth_stencil     { assert_eq!(texture.write_locks(), a) }
[INFO] [stderr] 31 +         while let Some(texture) = &self.depth_stencil     { assert_eq!(texture.write_locks(), a) }
[INFO] [stderr]    |
[INFO] [stderr] help: consider using `if let` to clear intent
[INFO] [stderr]    |
[INFO] [stderr] 31 -         for texture in &self.depth_stencil     { assert_eq!(texture.write_locks(), a) }
[INFO] [stderr] 31 +         if let Some(texture) = &self.depth_stencil     { assert_eq!(texture.write_locks(), a) }
[INFO] [stderr]    |
[INFO] [stderr] 
[INFO] [stderr] warning: for loop over a `&Option`. This is more readably written as an `if let` statement
[INFO] [stderr]   --> src/framebuffer.rs:38:18
[INFO] [stderr]    |
[INFO] [stderr] 38 |         for texture in &self.depth_stencil     { assert_eq!(texture.read_locks(), a) }
[INFO] [stderr]    |                        ^^^^^^^^^^^^^^^^^^^
[INFO] [stderr]    |
[INFO] [stderr] help: to check pattern in a loop use `while let`
[INFO] [stderr]    |
[INFO] [stderr] 38 -         for texture in &self.depth_stencil     { assert_eq!(texture.read_locks(), a) }
[INFO] [stderr] 38 +         while let Some(texture) = &self.depth_stencil     { assert_eq!(texture.read_locks(), a) }
[INFO] [stderr]    |
[INFO] [stderr] help: consider using `if let` to clear intent
[INFO] [stderr]    |
[INFO] [stderr] 38 -         for texture in &self.depth_stencil     { assert_eq!(texture.read_locks(), a) }
[INFO] [stderr] 38 +         if let Some(texture) = &self.depth_stencil     { assert_eq!(texture.read_locks(), a) }
[INFO] [stderr]    |
[INFO] [stderr] 
[INFO] [stderr] warning: for loop over a `&Option`. This is more readably written as an `if let` statement
[INFO] [stderr]   --> src/framebuffer.rs:45:18
[INFO] [stderr]    |
[INFO] [stderr] 45 |         for texture in &self.depth_stencil     { texture.acquire_write(); }
[INFO] [stderr]    |                        ^^^^^^^^^^^^^^^^^^^
[INFO] [stderr]    |
[INFO] [stderr] help: to check pattern in a loop use `while let`
[INFO] [stderr]    |
[INFO] [stderr] 45 -         for texture in &self.depth_stencil     { texture.acquire_write(); }
[INFO] [stderr] 45 +         while let Some(texture) = &self.depth_stencil     { texture.acquire_write(); }
[INFO] [stderr]    |
[INFO] [stderr] help: consider using `if let` to clear intent
[INFO] [stderr]    |
[INFO] [stderr] 45 -         for texture in &self.depth_stencil     { texture.acquire_write(); }
[INFO] [stderr] 45 +         if let Some(texture) = &self.depth_stencil     { texture.acquire_write(); }
[INFO] [stderr]    |
[INFO] [stderr] 
[INFO] [stderr] warning: for loop over a `&Option`. This is more readably written as an `if let` statement
[INFO] [stderr]   --> src/framebuffer.rs:50:18
[INFO] [stderr]    |
[INFO] [stderr] 50 |         for texture in &self.depth_stencil     { texture.release_write(); }
[INFO] [stderr]    |                        ^^^^^^^^^^^^^^^^^^^
[INFO] [stderr]    |
[INFO] [stderr] help: to check pattern in a loop use `while let`
[INFO] [stderr]    |
[INFO] [stderr] 50 -         for texture in &self.depth_stencil     { texture.release_write(); }
[INFO] [stderr] 50 +         while let Some(texture) = &self.depth_stencil     { texture.release_write(); }
[INFO] [stderr]    |
[INFO] [stderr] help: consider using `if let` to clear intent
[INFO] [stderr]    |
[INFO] [stderr] 50 -         for texture in &self.depth_stencil     { texture.release_write(); }
[INFO] [stderr] 50 +         if let Some(texture) = &self.depth_stencil     { texture.release_write(); }
[INFO] [stderr]    |
[INFO] [stderr] 
[INFO] [stderr] warning: for loop over a `&Option`. This is more readably written as an `if let` statement
[INFO] [stderr]   --> src/framebuffer.rs:55:18
[INFO] [stderr]    |
[INFO] [stderr] 55 |         for texture in &self.depth_stencil     { texture.acquire_read(); }
[INFO] [stderr]    |                        ^^^^^^^^^^^^^^^^^^^
[INFO] [stderr]    |
[INFO] [stderr] help: to check pattern in a loop use `while let`
[INFO] [stderr]    |
[INFO] [stderr] 55 -         for texture in &self.depth_stencil     { texture.acquire_read(); }
[INFO] [stderr] 55 +         while let Some(texture) = &self.depth_stencil     { texture.acquire_read(); }
[INFO] [stderr]    |
[INFO] [stderr] help: consider using `if let` to clear intent
[INFO] [stderr]    |
[INFO] [stderr] 55 -         for texture in &self.depth_stencil     { texture.acquire_read(); }
[INFO] [stderr] 55 +         if let Some(texture) = &self.depth_stencil     { texture.acquire_read(); }
[INFO] [stderr]    |
[INFO] [stderr] 
[INFO] [stderr] warning: for loop over a `&Option`. This is more readably written as an `if let` statement
[INFO] [stderr]   --> src/framebuffer.rs:60:18
[INFO] [stderr]    |
[INFO] [stderr] 60 |         for texture in &self.depth_stencil     { texture.release_read(); }
[INFO] [stderr]    |                        ^^^^^^^^^^^^^^^^^^^
[INFO] [stderr]    |
[INFO] [stderr] help: to check pattern in a loop use `while let`
[INFO] [stderr]    |
[INFO] [stderr] 60 -         for texture in &self.depth_stencil     { texture.release_read(); }
[INFO] [stderr] 60 +         while let Some(texture) = &self.depth_stencil     { texture.release_read(); }
[INFO] [stderr]    |
[INFO] [stderr] help: consider using `if let` to clear intent
[INFO] [stderr]    |
[INFO] [stderr] 60 -         for texture in &self.depth_stencil     { texture.release_read(); }
[INFO] [stderr] 60 +         if let Some(texture) = &self.depth_stencil     { texture.release_read(); }
[INFO] [stderr]    |
[INFO] [stderr] 
[INFO] [stderr] warning: `gavle` (lib) generated 20 warnings (run `cargo fix --lib -p gavle` to apply 8 suggestions)
[INFO] [stderr] warning: `gavle` (lib test) generated 20 warnings (20 duplicates)
[INFO] [stderr]     Finished `test` profile [unoptimized + debuginfo] target(s) in 0.07s
[INFO] [stderr]      Running unittests src/lib.rs (/opt/rustwide/target/debug/deps/gavle-029d32784858ea87)
[INFO] [stdout] 
[INFO] [stdout] running 3 tests
[INFO] [stdout] test info::tests::release ... ok
[INFO] [stdout] test info::tests::version ... ok
[INFO] [stdout] test info::tests::profile ... ok
[INFO] [stdout] 
[INFO] [stdout] test result: ok. 3 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s
[INFO] [stdout] 
[INFO] [stderr]    Doc-tests gavle
[INFO] [stdout] 
[INFO] [stdout] running 1 test
[INFO] [stdout] test src/shader.rs - shader::instance_shaders (line 45) ... FAILED
[INFO] [stdout] 
[INFO] [stdout] failures:
[INFO] [stdout] 
[INFO] [stdout] ---- src/shader.rs - shader::instance_shaders (line 45) stdout ----
[INFO] [stdout] error: cannot find macro `instance_shaders` in this scope
[INFO] [stdout]   --> src/shader.rs:46:1
[INFO] [stdout]    |
[INFO] [stdout] 46 | instance_shaders! {
[INFO] [stdout]    | ^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] error: aborting due to 1 previous error
[INFO] [stdout] 
[INFO] [stdout] Couldn't compile the test.
[INFO] [stdout] 
[INFO] [stdout] failures:
[INFO] [stdout]     src/shader.rs - shader::instance_shaders (line 45)
[INFO] [stdout] 
[INFO] [stdout] test result: FAILED. 0 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.09s
[INFO] [stdout] 
[INFO] [stderr] error: doctest failed, to rerun pass `--doc`
[INFO] running `Command { std: "docker" "inspect" "ba3b9345be999423255a08882e09ef70230a3ba18bf53eee093e36111e0225cc", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "ba3b9345be999423255a08882e09ef70230a3ba18bf53eee093e36111e0225cc", kill_on_drop: false }`
[INFO] [stdout] ba3b9345be999423255a08882e09ef70230a3ba18bf53eee093e36111e0225cc
