squiid/rust
1
0
Fork 0
Code Issues Pull requests Projects Activity

Fix some more dogfood issues

Browse source
This commit is contained in:
Oliver Schneider 2017-08-01 13:05:57 +02:00
parent a5fe4bed0e
commit 6bdb597a9f
1 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
clippy_lints/src/literal_digit_grouping.rs
View file

@ -147,7 +147,7 @@ impl<'a> DigitInfo<'a> {
if self.digits.contains('.') {
let mut parts = self.digits.split('.');
let int_part_hint = parts.next()
.unwrap()
.expect("split always returns at least one element")
.chars()
.rev()
.filter(|&c| c != '_')
@ -158,7 +158,7 @@ impl<'a> DigitInfo<'a> {
.collect::<Vec<String>>()
.join("_");
let frac_part_hint = parts.next()
.unwrap()
.expect("already checked that there is a `.`")
.chars()
.filter(|&c| c != '_')
.collect::<Vec<_>>()
@ -329,7 +329,7 @@ impl LiteralDigitGrouping {
.windows(2)
.all(|ps| ps[1] - ps[0] == group_size + 1)
// number of digits to the left of the last group cannot be bigger than group size.
&& (digits.len() - underscore_positions.last().unwrap() <= group_size + 1);
&& (digits.len() - underscore_positions.last().expect("there's at least one element") <= group_size + 1);
if !consistent {
return Err(WarningType::InconsistentDigitGrouping);