Re: [PATCH] rust: clean Rust 1.86.0 new `clippy::needless_continue` cases

From: Miguel Ojeda
Date: Wed Apr 02 2025 - 06:56:01 EST


On Wed, Apr 2, 2025 at 11:18 AM Danilo Krummrich <dakr@xxxxxxxxxx> wrote:
>
> Not related to the patch itself: Don't we need to disable new lints anyways?
> Otherwise we'd get warning when compiling older kernel with newer compilers /
> linters, no?

So far we are cleaning them up as they arrive (and backporting) and
deciding if we want to keep them or not etc. My plan is to eventually
have an explicit set of the ones we really, really want to keep clean,
since at some point it will be too much to clean (if the rate of
changes on Clippy is steady), and leave the rest to something like
`W=`.

In any case, `CLIPPY=1` is opt-in and the future can always give us
issues: even when we set an explicit list, sometimes lints have added
new cases (like in this patch), so we will need cleanups from time to
time either way.

Relatedly, we discussed at FOSDEM having Clippy be potentially
unconditionally used, but that would require changes on their side
first.

Cheers,
Miguel