diff options
| author | Corey Farwell <coreyf@rwell.org> | 2019-03-25 23:16:40 +0100 |
|---|---|---|
| committer | Corey Farwell <coreyf@rwell.org> | 2019-03-25 23:29:49 +0100 |
| commit | 28c602a94ebe15b45cfda59a4681d6916901aa40 (patch) | |
| tree | ebf90a1c6f7aa33d6749d805dafa12a725fd5997 | |
| parent | 4c27fb19ba15a2e45485e601a79914c6280196b0 (diff) | |
Utilize `?` instead of `return None`.
| -rw-r--r-- | src/libcore/num/flt2dec/mod.rs | 6 | ||||
| -rw-r--r-- | src/librustc/traits/util.rs | 6 | ||||
| -rw-r--r-- | src/librustc_typeck/check/mod.rs | 5 |
3 files changed, 4 insertions, 13 deletions
diff --git a/src/libcore/num/flt2dec/mod.rs b/src/libcore/num/flt2dec/mod.rs index defd4247f4e..c9a9375ec59 100644 --- a/src/libcore/num/flt2dec/mod.rs +++ b/src/libcore/num/flt2dec/mod.rs @@ -239,10 +239,8 @@ impl<'a> Formatted<'a> { let mut written = self.sign.len(); for part in self.parts { - match part.write(&mut out[written..]) { - Some(len) => { written += len; } - None => { return None; } - } + let len = part.write(&mut out[written..])?; + written += len; } Some(written) } diff --git a/src/librustc/traits/util.rs b/src/librustc/traits/util.rs index c3223f0cd63..90f62a4d132 100644 --- a/src/librustc/traits/util.rs +++ b/src/librustc/traits/util.rs @@ -292,11 +292,7 @@ impl<'cx, 'gcx, 'tcx> Iterator for SupertraitDefIds<'cx, 'gcx, 'tcx> { type Item = DefId; fn next(&mut self) -> Option<DefId> { - let def_id = match self.stack.pop() { - Some(def_id) => def_id, - None => { return None; } - }; - + let def_id = self.stack.pop()?; let predicates = self.tcx.super_predicates_of(def_id); let visited = &mut self.visited; self.stack.extend( diff --git a/src/librustc_typeck/check/mod.rs b/src/librustc_typeck/check/mod.rs index 94f76b03a64..b11bd9c2408 100644 --- a/src/librustc_typeck/check/mod.rs +++ b/src/librustc_typeck/check/mod.rs @@ -5318,10 +5318,7 @@ impl<'a, 'gcx, 'tcx> FnCtxt<'a, 'gcx, 'tcx> { ) -> Option<Span> { // Be helpful when the user wrote `{... expr;}` and // taking the `;` off is enough to fix the error. - let last_stmt = match blk.stmts.last() { - Some(s) => s, - None => return None, - }; + let last_stmt = blk.stmts.last()?; let last_expr = match last_stmt.node { hir::StmtKind::Semi(ref e) => e, _ => return None, |
