Lint bit width

[alv-around]

Fixes #16876

Current state:

• suggests change of T::BITS - x.leading_zeros() for uints
• suggests change of T::BITS - x.leading_zeros() for NonZero

Description

rust 1.97 introduces the method bit_width() for uints and NonZero (docs), this will make the manual computation redundant and unnecessary.

Example

let x: u32 = b'101';
let bit_width = u32::BITS - x.leading_zeros();

Can be replaced with:

let x: u32 = b'101';
let bit_width = x.bit_width();

changelog: [manual_bit_width]: Added a lint to detect bit_width implementations.

I have

• Followed [lint naming conventions][lint_naming]
• Added passing UI tests (including committed .stderr file)
• cargo test passes locally
• Executed cargo dev update_lints
• Added lint documentation
• Run cargo dev fmt

[rustbot]

r? @dswij

rustbot has assigned @dswij.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

• Owners of files modified in this PR: 7 candidates
• 7 candidates expanded to 7 candidates
• Random selection from Jarcho, dswij, llogiq

[alv-around]

this two lines could be replaced by the function int_ty_to_nbits util function. But the functions should be move from clippy_lints/src/casts/utils.rs to clippy_utils and I was unsure whether this is desired. I think that would be a good choice overall, as ty::Uint(uint_ty) => uint_ty.bitwidth(), return None for usize.

View changes since the review

[alv-around]

Same as in the previous comment

View changes since the review