[pbs-devel] [PATCH proxmox-backup v2 00/14] various clippy fixes

Dominik Csapak d.csapak at proxmox.com
Mon Apr 19 08:59:36 CEST 2021


On 4/18/21 16:26, Dietmar Maurer wrote:
> Some of those changes may make sense, but it is really
> a PITA to review them. Every change can break the code an introduce new
> errors (which already happened in the past).
> 

any examples for those errors? (just for my curiosity)

can i do anything to make reviewing easier?

> On the other side, It is hard to see any advantages for
> most of those changes.
> 

IMHO, most changes do make the code better and/or more
readable (e.g. nested ifs, complex types, safety annotation,
absurd extreme comparisons), any specific ones you
dislike?

if there are some classes of warnings/errors that do not
make sense, it would probably be good if we introduce some
custom clippy configs in our repos





More information about the pbs-devel mailing list