Less error-prone naming

This commit is contained in:
Aleksey Kladov 2020-06-24 16:58:49 +02:00
parent ddc5a3e567
commit 6937bcb67d
4 changed files with 12 additions and 21 deletions

View file

@ -67,8 +67,8 @@ pub(crate) fn load(
// wait until Vfs has loaded all roots // wait until Vfs has loaded all roots
for task in receiver { for task in receiver {
match task { match task {
vfs::loader::Message::Progress { n_entries_done, n_entries_total } => { vfs::loader::Message::Progress { n_done, n_total } => {
if n_entries_done == n_entries_total { if n_done == n_total {
break; break;
} }
} }

View file

@ -305,18 +305,12 @@ fn loop_turn(
} }
} }
} }
vfs::loader::Message::Progress { n_entries_total, n_entries_done } => { vfs::loader::Message::Progress { n_total, n_done } => {
if n_entries_done == n_entries_total { if n_done == n_total {
loop_state.status = Status::Ready; loop_state.status = Status::Ready;
became_ready = true; became_ready = true;
} }
report_progress( report_progress(loop_state, &connection.sender, n_done, n_total, "roots scanned")
loop_state,
&connection.sender,
n_entries_done,
n_entries_total,
"roots scanned",
)
} }
}, },
Event::CheckWatcher(task) => on_check_task(task, global_state, task_sender)?, Event::CheckWatcher(task) => on_check_task(task, global_state, task_sender)?,

View file

@ -89,8 +89,8 @@ impl LoaderActor {
match event { match event {
Event::Message(msg) => match msg { Event::Message(msg) => match msg {
Message::Config(config) => { Message::Config(config) => {
let n_entries_total = config.load.len(); let n_total = config.load.len();
self.send(loader::Message::Progress { n_entries_total, n_entries_done: 0 }); self.send(loader::Message::Progress { n_total, n_done: 0 });
self.unwatch_all(); self.unwatch_all();
self.config.clear(); self.config.clear();
@ -99,10 +99,7 @@ impl LoaderActor {
let watch = config.watch.contains(&i); let watch = config.watch.contains(&i);
let files = self.load_entry(entry, watch); let files = self.load_entry(entry, watch);
self.send(loader::Message::Loaded { files }); self.send(loader::Message::Loaded { files });
self.send(loader::Message::Progress { self.send(loader::Message::Progress { n_total, n_done: i + 1 });
n_entries_total,
n_entries_done: i + 1,
});
} }
self.config.sort_by(|x, y| x.0.cmp(&y.0)); self.config.sort_by(|x, y| x.0.cmp(&y.0));
} }

View file

@ -16,7 +16,7 @@ pub struct Config {
} }
pub enum Message { pub enum Message {
Progress { n_entries_total: usize, n_entries_done: usize }, Progress { n_total: usize, n_done: usize },
Loaded { files: Vec<(AbsPathBuf, Option<Vec<u8>>)> }, Loaded { files: Vec<(AbsPathBuf, Option<Vec<u8>>)> },
} }
@ -56,10 +56,10 @@ impl fmt::Debug for Message {
Message::Loaded { files } => { Message::Loaded { files } => {
f.debug_struct("Loaded").field("n_files", &files.len()).finish() f.debug_struct("Loaded").field("n_files", &files.len()).finish()
} }
Message::Progress { n_entries_total, n_entries_done } => f Message::Progress { n_total, n_done } => f
.debug_struct("Progress") .debug_struct("Progress")
.field("n_entries_total", n_entries_total) .field("n_total", n_total)
.field("n_entries_done", n_entries_done) .field("n_done", n_done)
.finish(), .finish(),
} }
} }