search

shnewto / bnf

Parse BNF grammar definitions
MIT License
256 stars 22 forks source link

Clean iter mut pattern #110

Closed CrockAgile closed 1 year ago

CrockAgile commented 1 year ago

This is not a bug fix, and criterion saw no performance change. I saw this pattern in some code the other day and thought it was more clear. Maybe subjective!

coveralls commented 1 year ago

Coverage Status

Coverage decreased (-0.0001%) to 92.873% when pulling 2b8e7941df04aaf75bf56f977954c178bc42199c on clean-iter-mut into d37b4c2bbd46afdf278883975d74063c4bf6e25b on main.

shnewto commented 1 year ago

This is cool! Totally thinkin on where else I can apply the same pattern now too 🤩 Weird new clippy stuff from these "Unchanged files with check annotations" reports. If you feel like tackling those sometime I'd be glad for it but we could also make an issue w the help wanted / good first issue tag to knock out clippy stuff like this