Skip to content

Add missing semicolon to satisfy clippy#3118

Merged
hecrj merged 1 commit intoiced-rs:masterfrom
mfreeborn:appease-clippy
Nov 30, 2025
Merged

Add missing semicolon to satisfy clippy#3118
hecrj merged 1 commit intoiced-rs:masterfrom
mfreeborn:appease-clippy

Conversation

@mfreeborn
Copy link
Contributor

Just a trivial PR

@mfreeborn
Copy link
Contributor Author

The other one I'm seeing is

warning: use of `default` to create a unit struct
   --> wgpu/src/layer.rs:386:21
    |
386 |             images: image::Batch::default(),
    |                     ^^^^^^^^^^^^-----------
    |                                 |
    |                                 help: remove this call to `default`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/rust-1.91.0/index.html#default_constructed_unit_structs
    = note: `#[warn(clippy::default_constructed_unit_structs)]` on by default

But image::Batch is defined as pub type Batch = Vec<Image>, so I don't understand how that works...

@mfreeborn
Copy link
Contributor Author

(I admit I thought a bit more would go in this PR when I created it; I realise it's a bit pointless on it's own and you'll just sort out clippy and whatever yourself as and when required)

@hecrj hecrj added this to the 0.14 milestone Nov 30, 2025
@hecrj hecrj added improvement An internal improvement fix labels Nov 30, 2025
@hecrj hecrj enabled auto-merge November 30, 2025 18:25
@hecrj hecrj merged commit 87c3d76 into iced-rs:master Nov 30, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix improvement An internal improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants