ndarray icon indicating copy to clipboard operation
ndarray copied to clipboard

fix workaround false positive clippy lint `reversed_empty_ranges`

Open ModProg opened this issue 2 years ago • 3 comments

As the rust-clippy issue https://github.com/rust-lang/rust-clippy/issues/5808 doesn't seam to make much progress I propose to just allow the lint for s![] macro invocations for now.

The let = ... is necessary, because attributes are not allowed on expressions currently (https://github.com/rust-lang/rust/issues/15701).

ModProg avatar Apr 23 '23 07:04 ModProg

@bluss removed the depricated zip

ModProg avatar Apr 30 '23 17:04 ModProg

Can we hold off on removing the deprecated zip until it can be replaced by std::iter::zip? That's after all the whole point of it, to use the function version :slightly_smiling_face:

(Function zip is the best :slightly_smiling_face: )

bluss avatar Apr 30 '23 20:04 bluss

Yeah no problem, only did so because the ci was failing because of it.

ModProg avatar Apr 30 '23 22:04 ModProg