diff options
| author | Alexey Shmalko <rasen.dubi@gmail.com> | 2019-05-03 22:45:59 +0300 |
|---|---|---|
| committer | Alexey Shmalko <rasen.dubi@gmail.com> | 2019-05-03 22:45:59 +0300 |
| commit | bacf792f6f8b3ae2e94930eb11faeaa49f081549 (patch) | |
| tree | ae5152ed5538f5be1e86fa813e46294145b8d783 | |
| parent | 1937bf26268d88fabd52873a2bb0ef60e078422f (diff) | |
tidy: Extract `let mut part` out of `parts` block in `version.rs`
| -rw-r--r-- | src/tools/tidy/src/features/version.rs | 12 |
1 files changed, 5 insertions, 7 deletions
diff --git a/src/tools/tidy/src/features/version.rs b/src/tools/tidy/src/features/version.rs index 6027e7d35e2..1ce9fe127dd 100644 --- a/src/tools/tidy/src/features/version.rs +++ b/src/tools/tidy/src/features/version.rs @@ -31,15 +31,13 @@ impl FromStr for Version { fn from_str(s: &str) -> Result<Self, Self::Err> { let mut iter = s.split('.').map(|part| Ok(part.parse()?)); - let parts = { - let mut part = || { - iter.next() - .unwrap_or(Err(ParseVersionError::WrongNumberOfParts)) - }; - - [part()?, part()?, part()?] + let mut part = || { + iter.next() + .unwrap_or(Err(ParseVersionError::WrongNumberOfParts)) }; + let parts = [part()?, part()?, part()?]; + if let Some(_) = iter.next() { // Ensure we don't have more than 3 parts. return Err(ParseVersionError::WrongNumberOfParts); |
