From cfbf47b0023585a30d825b5c5da8e2fb9d6fc337 Mon Sep 17 00:00:00 2001
From: Bernardo <berublan@gmail.com>
Date: Wed, 23 Jan 2019 20:43:35 +0100
Subject: review fixes

---
 crates/ra_vfs/src/io.rs    | 29 +++++++++--------------------
 crates/ra_vfs/src/lib.rs   |  1 -
 crates/ra_vfs/tests/vfs.rs |  2 +-
 3 files changed, 10 insertions(+), 22 deletions(-)

(limited to 'crates')

diff --git a/crates/ra_vfs/src/io.rs b/crates/ra_vfs/src/io.rs
index 335f4f2e1..83a021c2f 100644
--- a/crates/ra_vfs/src/io.rs
+++ b/crates/ra_vfs/src/io.rs
@@ -50,7 +50,6 @@ pub enum TaskResult {
     AddRoot(AddRootResult),
     HandleChange(WatcherChange),
     LoadChange(WatcherChangeData),
-    NoOp,
 }
 
 impl fmt::Debug for TaskResult {
@@ -59,7 +58,6 @@ impl fmt::Debug for TaskResult {
             TaskResult::AddRoot(..) => f.write_str("TaskResult::AddRoot(..)"),
             TaskResult::HandleChange(c) => write!(f, "TaskResult::HandleChange({:?})", c),
             TaskResult::LoadChange(c) => write!(f, "TaskResult::LoadChange({:?})", c),
-            TaskResult::NoOp => f.write_str("TaskResult::NoOp"),
         }
     }
 }
@@ -78,7 +76,7 @@ impl Worker {
             thread_worker::spawn("vfs", 128, move |input_receiver, output_sender| {
                 input_receiver
                     .into_iter()
-                    .map(|t| handle_task(t, &watcher_clone))
+                    .filter_map(|t| handle_task(t, &watcher_clone))
                     .try_for_each(|it| output_sender.send(it))
                     .unwrap()
             });
@@ -118,18 +116,12 @@ fn watch(
     filter_entry: &RootFilter,
     emit_for_existing: bool,
 ) {
-    let mut watcher = watcher.lock();
-    let watcher = match *watcher {
-        Some(ref mut w) => w,
-        None => {
-            // watcher dropped or couldn't start
-            return;
-        }
-    };
-    watcher.watch_recursive(dir, filter_entry, emit_for_existing)
+    if let Some(watcher) = watcher.lock().as_mut() {
+        watcher.watch_recursive(dir, filter_entry, emit_for_existing)
+    }
 }
 
-fn handle_task(task: Task, watcher: &Arc<Mutex<Option<Watcher>>>) -> TaskResult {
+fn handle_task(task: Task, watcher: &Arc<Mutex<Option<Watcher>>>) -> Option<TaskResult> {
     match task {
         Task::AddRoot {
             root,
@@ -145,22 +137,19 @@ fn handle_task(task: Task, watcher: &Arc<Mutex<Option<Watcher>>>) -> TaskResult
                 nested_roots.as_slice(),
             );
             log::debug!("... loaded {}", path.as_path().display());
-            TaskResult::AddRoot(AddRootResult { root, files })
+            Some(TaskResult::AddRoot(AddRootResult { root, files }))
         }
         Task::HandleChange(change) => {
             // forward as is because Vfs has to decide if we should load it
-            TaskResult::HandleChange(change)
+            Some(TaskResult::HandleChange(change))
         }
         Task::LoadChange(change) => {
             log::debug!("loading {:?} ...", change);
-            match load_change(change) {
-                Some(data) => TaskResult::LoadChange(data),
-                None => TaskResult::NoOp,
-            }
+            load_change(change).map(TaskResult::LoadChange)
         }
         Task::Watch { dir, root_filter } => {
             watch(watcher, &dir, root_filter.as_ref(), true);
-            TaskResult::NoOp
+            None
         }
     }
 }
diff --git a/crates/ra_vfs/src/lib.rs b/crates/ra_vfs/src/lib.rs
index bcff7928b..f6b45c18a 100644
--- a/crates/ra_vfs/src/lib.rs
+++ b/crates/ra_vfs/src/lib.rs
@@ -259,7 +259,6 @@ impl Vfs {
                     }
                 }
             },
-            TaskResult::NoOp => {}
         }
     }
 
diff --git a/crates/ra_vfs/tests/vfs.rs b/crates/ra_vfs/tests/vfs.rs
index d3271570a..bf44e97c5 100644
--- a/crates/ra_vfs/tests/vfs.rs
+++ b/crates/ra_vfs/tests/vfs.rs
@@ -117,7 +117,7 @@ fn test_vfs_works() -> std::io::Result<()> {
 
     fs::create_dir_all(dir.path().join("a/sub1/sub2")).unwrap();
     fs::write(dir.path().join("a/sub1/sub2/new.rs"), "new hello").unwrap();
-    process_tasks(&mut vfs, 4);
+    process_tasks(&mut vfs, 3);
     assert_match!(
         vfs.commit_changes().as_slice(),
         [VfsChange::AddFile { text, path, .. }],
-- 
cgit v1.2.3