miam-miam
miam-miam
Hmmm it looks like the exact same lint was added directly to rustc for release 1.64.0 (https://github.com/rust-lang/rust/pull/98580). What were the chances of that?
Hmmm so if I've understood correctly we should have a lint that prevents you from writing this? ```rust println!("hello {world} {}", world = "world"); ```
I don't know what rustc version clippy is using but it looks like my two errors are due to these two PRs not being merged in yet: https://github.com/rust-lang/rust/pull/99263 and https://github.com/rust-lang/rust/pull/99480....
The changes do not currently pass on my local machine as the toolchain version clippy uses does not include the changes I need for the lint to work correctly. However...
That's awesome, that does not sound all too easy so I wish you the best of luck on that! I added the test cases to my code which look like...
Nice! Successfully rebased, @dswij feel free to review me whenever you are free.
Hmmm I'm not sure that is a great idea since we don't know when that PR will land and this PR does not really benefit from the late pass in...
Ah well fortunately for you, all those things are directly exposed by rustc. All the arg, width and precision spans are given, as you can see here (https://github.com/miam-miam100/rust-clippy/blob/631525a21fa3b4bd0c495da11c3b1c0eec84c5ff/clippy_lints/src/write.rs#L543). As for...
Right that's great thanks, I'm currently on holiday so I'll do that when I get back. (a week's time)
@dswij All squashed!