search

projectfluent / fluent-rs

Rust implementation of Project Fluent
https://projectfluent.org
Apache License 2.0
1.1k stars 98 forks source link

Apply new clippy lint fixes #375

Open alerque opened 4 days ago

alerque commented 4 days ago

The first commit here has some obvious fixes for convoluted code that has idiomatic Rust alternatives.

The second commit is for using lifetime elision instead of annotations. I understand this one is a bit more controversion and some projects have opted to ignore this lint instead. Personally I think I'm fine with it, I'd rather see annotations where they are meaningful and need to be included in reading the code to grok it, but if anybody here has issues with that I'm happy to consider ignoring these.

alerque commented 4 days ago

@waywardmonkeys Any idea off the top of your head how to deal with the other new lint not covered here yet involving mutable references to mutable static?