-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
deny clippy::type_complexity #401
Conversation
c89e029
to
14c4eb7
Compare
61a07e5
to
f3b0634
Compare
f3b0634
to
522d56e
Compare
have_gex: bool, | ||
gex_info: &GexInfo, | ||
rsi: &ColInfo, | ||
sr: &mut Vec<(Vec<String>, Vec<Vec<String>>, Vec<Vec<u8>>, usize)>, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good that we got rid of two unused fields here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And gigantic ones at that!
let mut bads = 0; | ||
let mut denom = 0; | ||
for r in results { | ||
bads += r.2; | ||
denom += r.3; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, duplicated computation. Good catch!
Removes the allow exception for
clippy::type_complexity
and fixes all errors. I was mostly able to replace anonymous tuples with structs, but one section relied on some existing tuple-specific logic so I created a type alias and moved on.Stacked on another feature branch, will rebase once underlying branch merges.