search

dfrg / swash

Font introspection, complex text shaping and glyph rendering.
Apache License 2.0
584 stars 34 forks source link

Clippy fixes #11

Closed ajtribick closed 2 years ago

ajtribick commented 2 years ago

This fixes the Clippy warnings in the project. I realise this has a lot of files changed, so I split it into a commit per directory within src. If you prefer, I can raise this with each commit in its own pull request.

There are a couple of API changes (e.g. methods taking self vs &self, and replacing an inherent to_string with impl core::fmt::Display) as a result of fixing lints on various to_xyz methods, though these are typically on types that are already Copy.

I have not run cargo fmt on the output, to avoid too much noise in the changes.

dfrg commented 2 years ago

Wow! Thanks so much for this. I've read over the diffs and it all looks good to me so I'm going to go ahead and merge.