diff --git a/crates/ra_vfs/src/io.rs b/crates/ra_vfs/src/io.rs index 257f1bf538..c467605835 100644 --- a/crates/ra_vfs/src/io.rs +++ b/crates/ra_vfs/src/io.rs @@ -4,38 +4,47 @@ use std::{ thread::JoinHandle, }; -use walkdir::WalkDir; +use walkdir::{DirEntry, WalkDir}; use crossbeam_channel::{Sender, Receiver}; -use thread_worker::{WorkerHandle, Worker}; +use thread_worker::{WorkerHandle}; -#[derive(Debug)] -pub struct FileEvent { - pub path: PathBuf, - pub kind: FileEventKind, +use crate::VfsRoot; + +pub(crate) enum Task { + ScanRoot { + root: VfsRoot, + path: PathBuf, + filter: Box bool + Send>, + }, } #[derive(Debug)] -pub enum FileEventKind { +pub(crate) struct FileEvent { + pub(crate) path: PathBuf, + pub(crate) kind: FileEventKind, +} + +#[derive(Debug)] +pub(crate) enum FileEventKind { Add(String), } -pub(crate) type FsWorker = Worker)>; +pub(crate) type Worker = thread_worker::Worker)>; -pub(crate) fn start() -> (FsWorker, WorkerHandle) { - thread_worker::spawn::), _>( - "vfs", - 128, - |input_receiver, output_sender| { - input_receiver - .map(|path| { - log::debug!("loading {} ...", path.as_path().display()); - let events = load_root(path.as_path()); - log::debug!("... loaded {}", path.as_path().display()); - (path, events) - }) - .for_each(|it| output_sender.send(it)) - }, - ) +pub(crate) fn start() -> (Worker, WorkerHandle) { + thread_worker::spawn("vfs", 128, |input_receiver, output_sender| { + input_receiver + .map(handle_task) + .for_each(|it| output_sender.send(it)) + }) +} + +fn handle_task(task: Task) -> (PathBuf, Vec) { + let Task::ScanRoot { path, .. } = task; + log::debug!("loading {} ...", path.as_path().display()); + let events = load_root(path.as_path()); + log::debug!("... loaded {}", path.as_path().display()); + (path, events) } fn load_root(path: &Path) -> Vec { diff --git a/crates/ra_vfs/src/lib.rs b/crates/ra_vfs/src/lib.rs index d4ba2cb455..8ce6b6ee0a 100644 --- a/crates/ra_vfs/src/lib.rs +++ b/crates/ra_vfs/src/lib.rs @@ -2,8 +2,8 @@ //! //! When doing analysis, we don't want to do any IO, we want to keep all source //! code in memory. However, the actual source code is stored on disk, so you -//! need to get it into the memory in the first place somehow. VFS is the //! component which does this. +//! need to get it into the memory in the first place somehow. VFS is the //! //! It also is responsible for watching the disk for changes, and for merging //! editor state (modified, unsaved files) with disk state. @@ -23,11 +23,10 @@ use std::{ }; use relative_path::RelativePathBuf; -use thread_worker::{WorkerHandle, Worker}; +use thread_worker::{WorkerHandle}; use crate::{ arena::{ArenaId, Arena}, - io::{FileEvent, FsWorker}, }; /// `RootFilter` is a predicate that checks if a file can belong to a root. If @@ -87,7 +86,7 @@ struct Vfs { roots: Arena, files: Arena, // pending_changes: Vec, - worker: FsWorker, + worker: io::Worker, worker_handle: WorkerHandle, }