[codeflo]

A very low-effort way to learn good Rust patterns is to put

  #![warn(clippy::all)]

at the top of your crate’s entrypoint. This enables Rust’s default linter. It’s a lot more friendly and focused on good design than you might expect, often suggesting more elegant alternatives. Plus, many of its suggestions can be applied automatically in an environment like VS Code + rust-analyzer plugin.

[tasn]

Thanks a lot for the suggestion, I can't believe I didn't know about this. However I just tried it and I can't get it to work.

I added this to the top of https://github.com/etesync/etebase-rs/blob/master/src/lib.rs and then ran `cargo clippy`

    #![warn(clippy::all)]

    // Should fail https://rust-lang.github.io/rust-clippy/master/index.html#float_cmp
    pub fn bool_test(x: f32, y: f32) -> bool {
        x == y
    }

Any idea what's missing? Why is it not failing?

Edit: I know the above example is bad code, that's the point. I want clippy to complain about it but it doesn't.

[badtuple]

Does it work if you run `cargo clean` and then `cargo clippy` again? Clippy runs it's lints in an early pass of the compiler/checker. Many IDEs/editors will automatically `cargo check` under the hood to grab errors. Then when you run `cargo clippy`, that part of the compilation is already cached and so clippy doesn't give you any output :(

To my knowledge (it's been a while since I looked) fixing this behavior is blocked on cargo stabilizing something and has been for literal years.

That point of frustration aside, it's worth it...Clippy is an absolutely amazing piece of software. Both for pedagogy and normal development.

EDIT: Just dug up the issue. If you're on nightly you can use `cargo clippy -Z unstable-options` to avoid the clean/rebuild. Hopefully stuff gets stabilized soon. Here's the issue for reference: https://github.com/rust-lang/rust-clippy/issues/4612

[tasn]

That fixed it, thanks a lot!

[vbrandl]

Instead of `cargo clean`, `touch src/main.rs` or `touch src/lib.rs` (or actually touching any source file and thereby changing the `last modified` date to now) will have the same effect. That's what I've been using.

[chewbacha]

I’m not sure and can’t check at the moment; but maybe the functions not called so it’s being ignored?

[tasn]

It's public in my lib, so it's exported. Though I also tried calling it from my code, it still doesn't trigger a warning. :|

[conradludgate]

Comparing floats by equality is a dangerous pattern. It's easy for small precision errors to occur. You should instead check that they are close enough to each other, using an epsilon that you find appropriate, perhaps 1e-10. `(x - y).abs() < epsilon` should do the trick

[tasn]

I know, it's an example I was hoping clippy would catch in order for it to fail so I know it works. Read what I wrote...

[wyldfire]

It's in fact the dangerous pattern used to illustrate the linter under discussion here.

[smnscu]

Float comparison is an anti-pattern. Use an epsilon instead. https://stackoverflow.com/questions/4915462/how-should-i-do-...

[tasn]

I know, it's an example I was hoping clippy would catch in order for it to fail so I know it works. Read what I wrote...