Summary

This PR fixes some instances of redundant or deprecated path prefixes in the codebase.

Motivation

Code style.

Some numeric constants are currently imported from core, such as core::u64::MAX or core::f64::EPSILON. Clippy suggests that these be changed to the newer associated constants on the types themselves, e.g. u64::MAX and f64::EPSILON. Also, method calls like u64::max_value() have likewise been replaced with u64::MAX.
Some types are currently fully qualified despite a matching import. These redundant prefixes have been removed.
Some paths are currently prefixed with :: unnecessarily, which has been removed.
Instances of -f64::INFINITY have been replaced with f64::NEG_INFINITY.

dhardy commented 3 months ago

Thanks for clearing up some historical mess now that we no longer support an ancient Rust version.

Is this still WIP?

MichaelOwenDyer commented 3 months ago

Was trying to rebase onto the latest changes on master and I think I may have royally screwed up the branch 💀

MichaelOwenDyer commented 3 months ago

Okay, I have no idea how to fix this so I'm going to close this PR and try again 🤞🏽