FP redundant_field_name: for loop

[matthiaskrgr]

I seen this in cargo a lot:
code like this

                for line in String::from_utf8_lossy(&dst[start..]).lines() {

triggers

warning: redundant field names in struct initialization
   --> src/cargo/util/process_builder.rs:240:58
    |
240 |                 for line in String::from_utf8_lossy(&dst[start..]).lines() {
    |                                                          ^^^^^ help: replace it with: `start`
    |
    = note: #[warn(redundant_field_names)] on by default
    = help: for further information visit https://rust-lang-nursery.github.io/rust-clippy/v0.0.212/index.html#redundant_field_names

which looks like complete bogus to me.
https://github.com/rust-lang/cargo/blob/master/src/cargo/util/process_builder.rs#L240

EDIT: 0.0.212

[matthiaskrgr]

Issue 3000 \o/ :tada:

[flip1995]

With the newest changes we got multiple problems with ranges (as macro expansions) back, that were fixed in previous PRs. (40349b2)

For this issue see especially #2507

We also got some other problems, where we want to look at ranges, but do not get to it, because we bail out earlier, because of a if in_macro(span) { return } check. I plan to open an issue summarizing these bugs, once Clippy is building (and tests are fixed) again.

[matthiaskrgr]

I got clippy to build and tests to pass at least locally with #3001 :)