From f9817da0c465d6a1d58e5e72e065fe167819fdbd Mon Sep 17 00:00:00 2001 From: Jag Talon Date: Mon, 6 Apr 2015 19:47:20 -0400 Subject: [PATCH] net/storage_task.rs: Fix remove_item Use `and_then` and remove `unwrap` instead of using a `map` as described in https://github.com/servo/servo/issues/5548#issuecomment-90254644. --- components/net/storage_task.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/components/net/storage_task.rs b/components/net/storage_task.rs index 215d451a9ab..8aeceb3dd77 100644 --- a/components/net/storage_task.rs +++ b/components/net/storage_task.rs @@ -133,9 +133,9 @@ impl StorageManager { fn remove_item(&mut self, sender: Sender>, url: Url, storage_type: StorageType, name: DOMString) { let origin = self.get_origin_as_string(url); let data = self.select_data_mut(storage_type); - let old_value = data.get_mut(&origin).map(|entry| { + let old_value = data.get_mut(&origin).and_then(|entry| { entry.remove(&name) - }).unwrap(); + }); sender.send(old_value).unwrap(); }