Skip to content

Conversation

Iluvmagick
Copy link

I've first noticed this in the DSL Builder struct, where the vec function takes a &mut self, while not even using self.
This prompted me to run cargo clippy -- -W clippy::needless_pass_by_ref_mut, and fix everything I could find.

@jonathanpwang
Copy link
Contributor

We prefer not to change these for security reasons. Some of the &mut don't need to be there but are warnings of underlying mutations at the compiler level.

@Iluvmagick Iluvmagick deleted the needless_mut branch October 10, 2025 04:54
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.

2 participants