13 ms·
Does that mean your code is annotated with 300+ instances of `#[allow(clippy::unwrap_used)]` et al?
by an_ko 1y ago
Does that mean your code is annotated with 300+ instances of `#[allow(clippy::unwrap_used)]` et al?
- evrimoztamur 1y agoIt was the first time I set it up, then I went through every single instance and refactored with the appropriate choice. It wasn't as tedious as you might imagine, and again, I really don't have the option of letting my game crash. I think the only legitimate uses are for direct indexing for tile maps etc. where I do bounds checking on two axes and know that it will map correctly. to the underlying memory (but that's `clippy::indexing_slicing`, I have 0 `clippy::unwrap_used` in my codebase now). If you begin a new project with these lints, you'll quickly train to write idiomatic Option/Result handling code by default.