Skip to content

Fix all clippy::arithmetic_side_effects in mimeparser - #8818

Open
link2xt wants to merge 2 commits into
mainfrom
link2xt/mimeparser-no-arithmetic-side-effects
Open

link2xt wants to merge 2 commits into
mainfrom
link2xt/mimeparser-no-arithmetic-side-effects

Conversation

@link2xt

@link2xt link2xt commented Oct 12, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

…height

This is a minor fix, worst case an image with very large dimensions will not be classified as a File.
Mostly removing `clippy::arithmetic_side_effects`.
…arser

self.parts.remove(i) can panic if "i" is out of range,
but now it is obvious that it will not because "i" is returned from position(),
and clippy is happy.
@link2xt
link2xt marked this pull request as ready for review October 12, 2026 20:12

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant