FR: redundant_field_names should distinguish between all and some fields matching

[taralx]

redundant_field_names fires on any field being a match. This impairs readability in cases like this:

let thingy = Thingy {
   calc: big + calculation,
   redundant: redundant,
   other: stuff.into(),
};

As you can see, if you apply the suggestion, you end up with a weird line that is different from the rest. I suggest that the lint be split such that it distinguishes between "all fields are redundant" and "some fields are redundant".

[jmmv]

Came here to upvote this FR. But I'd also add that the lint check should also ensure the style is consistent within a given initializer. I.e. if a field has a non-redundant value, then all fields should have explicit values given to them; and if all are redundant, then all should be removed. I think it'd be kind of awkward to have that be a separate lint check than this one, as otherwise the semantics of the two combined might get confusing.