diff options
| author | Erick Tryzelaar <erick.tryzelaar@gmail.com> | 2013-08-04 16:30:51 -0700 |
|---|---|---|
| committer | Erick Tryzelaar <erick.tryzelaar@gmail.com> | 2013-08-07 08:52:09 -0700 |
| commit | 19e17f54a02e484f1ab4fd809caa0aaf3f3d14bc (patch) | |
| tree | a8849f904f46e56f8ad718ab2e5c56947872e05c /src/libstd | |
| parent | 1e490813b017f99cb4385fe846d645efe5d62b62 (diff) | |
| download | rust-19e17f54a02e484f1ab4fd809caa0aaf3f3d14bc.tar.gz rust-19e17f54a02e484f1ab4fd809caa0aaf3f3d14bc.zip | |
std: removed option.take_map{,_default}
Diffstat (limited to 'src/libstd')
| -rw-r--r-- | src/libstd/option.rs | 14 | ||||
| -rw-r--r-- | src/libstd/rt/kill.rs | 10 |
2 files changed, 5 insertions, 19 deletions
diff --git a/src/libstd/option.rs b/src/libstd/option.rs index e43ff65da5e..66b30d8dd03 100644 --- a/src/libstd/option.rs +++ b/src/libstd/option.rs @@ -241,20 +241,6 @@ impl<T> Option<T> { util::replace(self, None) } - /// As `map_move`, but swaps a None into the original option rather - /// than consuming it by-value. - #[inline] - pub fn take_map<U>(&mut self, blk: &fn(T) -> U) -> Option<U> { - self.take().map_move(blk) - } - - /// As `map_move_default`, but swaps a None into the original option - /// rather than consuming it by-value. - #[inline] - pub fn take_map_default<U> (&mut self, def: U, blk: &fn(T) -> U) -> U { - self.take().map_move_default(def, blk) - } - /// Apply a function to the contained value or do nothing. /// Returns true if the contained value was mutated. pub fn mutate(&mut self, f: &fn(T) -> T) -> bool { diff --git a/src/libstd/rt/kill.rs b/src/libstd/rt/kill.rs index d90ad07650d..789c7531eca 100644 --- a/src/libstd/rt/kill.rs +++ b/src/libstd/rt/kill.rs @@ -405,7 +405,7 @@ impl KillHandle { others.take().map_move_default(true, |f| f()) && { let mut inner = this.take().unwrap(); (!inner.any_child_failed) && - inner.child_tombstones.take_map_default(true, |f| f()) + inner.child_tombstones.take().map_move_default(true, |f| f()) } } } @@ -493,7 +493,7 @@ impl Death { { use util; util::ignore(group); } // Step 1. Decide if we need to collect child failures synchronously. - do self.on_exit.take_map |on_exit| { + do self.on_exit.take().map_move |on_exit| { if success { // We succeeded, but our children might not. Need to wait for them. let mut inner = self.kill_handle.take_unwrap().unwrap(); @@ -501,7 +501,7 @@ impl Death { success = false; } else { // Lockless access to tombstones protected by unwrap barrier. - success = inner.child_tombstones.take_map_default(true, |f| f()); + success = inner.child_tombstones.take().map_move_default(true, |f| f()); } } on_exit(success); @@ -510,12 +510,12 @@ impl Death { // Step 2. Possibly alert possibly-watching parent to failure status. // Note that as soon as parent_handle goes out of scope, the parent // can successfully unwrap its handle and collect our reported status. - do self.watching_parent.take_map |mut parent_handle| { + do self.watching_parent.take().map_move |mut parent_handle| { if success { // Our handle might be None if we had an exit callback, and // already unwrapped it. But 'success' being true means no // child failed, so there's nothing to do (see below case). - do self.kill_handle.take_map |own_handle| { + do self.kill_handle.take().map_move |own_handle| { own_handle.reparent_children_to(&mut parent_handle); }; } else { |
