njsmith / posy

289 stars 17 forks source link

fix: address clippy::unnecessary_unwrap #20

Closed rockstar closed 1 year ago

rockstar commented 1 year ago

This patch addresses the clippy::unnecessary_unwrap in the one case where it was occurring. In this case, I think just adding a local exception is the best for this issue, as the way to address this lint in the current form makes the code a little less clear.

njsmith commented 1 year ago

huh, weird lint.

njsmith commented 1 year ago

I mean, I didn't feel good writing that code, but yeah idk how you're supposed to do it better.

I guess by waiting for https://github.com/rust-lang/rust/issues/53667 to stabilize.