]> git.lizzy.rs Git - rust.git/commitdiff
internal: prepare to store OpQueue results in the queue itself
authorAleksey Kladov <aleksey.kladov@gmail.com>
Mon, 5 Apr 2021 17:49:00 +0000 (20:49 +0300)
committerAleksey Kladov <aleksey.kladov@gmail.com>
Mon, 5 Apr 2021 17:49:00 +0000 (20:49 +0300)
crates/rust-analyzer/src/global_state.rs
crates/rust-analyzer/src/op_queue.rs
crates/rust-analyzer/src/reload.rs

index 52c2497137f570c74a005cb7905038ab435e8ab1..8679c8599617f4c99a5c4cb5fdb638f44aae6f30 100644 (file)
@@ -81,10 +81,13 @@ pub(crate) struct GlobalState {
     pub(crate) status: Status,
     pub(crate) source_root_config: SourceRootConfig,
     pub(crate) proc_macro_client: Option<ProcMacroClient>,
+
     pub(crate) workspaces: Arc<Vec<ProjectWorkspace>>,
-    pub(crate) fetch_workspaces_queue: OpQueue<()>,
+    pub(crate) fetch_workspaces_queue: OpQueue<(), ()>,
+
     pub(crate) workspace_build_data: Option<BuildDataResult>,
-    pub(crate) fetch_build_data_queue: OpQueue<BuildDataCollector>,
+    pub(crate) fetch_build_data_queue: OpQueue<BuildDataCollector, ()>,
+
     latest_requests: Arc<RwLock<LatestRequests>>,
 }
 
index 761b9ad391d9f6399a075b371cf61cfd2089ac1d..f71b718bc08751324ec60f6f447933cd78ed53ec 100644 (file)
@@ -1,29 +1,37 @@
-//! Bookkeeping to make sure only one long-running operation is executed.
+//! Bookkeeping to make sure only one long-running operation is being executed
+//! at a time.
 
-pub(crate) struct OpQueue<D> {
-    op_scheduled: Option<D>,
+pub(crate) struct OpQueue<Args, Output> {
+    op_scheduled: Option<Args>,
     op_in_progress: bool,
+    last_op_result: Output,
 }
 
-impl<D> Default for OpQueue<D> {
+impl<Args, Output: Default> Default for OpQueue<Args, Output> {
     fn default() -> Self {
-        Self { op_scheduled: None, op_in_progress: false }
+        Self { op_scheduled: None, op_in_progress: false, last_op_result: Default::default() }
     }
 }
 
-impl<D> OpQueue<D> {
-    pub(crate) fn request_op(&mut self, data: D) {
+impl<Args, Output> OpQueue<Args, Output> {
+    pub(crate) fn request_op(&mut self, data: Args) {
         self.op_scheduled = Some(data);
     }
-    pub(crate) fn should_start_op(&mut self) -> Option<D> {
+    pub(crate) fn should_start_op(&mut self) -> Option<Args> {
         if self.op_in_progress {
             return None;
         }
         self.op_in_progress = self.op_scheduled.is_some();
         self.op_scheduled.take()
     }
-    pub(crate) fn op_completed(&mut self) {
+    pub(crate) fn op_completed(&mut self, result: Output) {
         assert!(self.op_in_progress);
         self.op_in_progress = false;
+        self.last_op_result = result;
+    }
+
+    #[allow(unused)]
+    pub(crate) fn last_op_result(&self) -> &Output {
+        &self.last_op_result
     }
 }
index 76fdbcdddb54fb93981ffded9398020acbd9de22..d12f891bbce60b2febcfca63141752f8911e4aca 100644 (file)
@@ -140,7 +140,7 @@ pub(crate) fn fetch_build_data_if_needed(&mut self) {
         });
     }
     pub(crate) fn fetch_build_data_completed(&mut self) {
-        self.fetch_build_data_queue.op_completed()
+        self.fetch_build_data_queue.op_completed(())
     }
 
     pub(crate) fn fetch_workspaces_request(&mut self) {
@@ -195,7 +195,7 @@ pub(crate) fn fetch_workspaces_if_needed(&mut self) {
         });
     }
     pub(crate) fn fetch_workspaces_completed(&mut self) {
-        self.fetch_workspaces_queue.op_completed()
+        self.fetch_workspaces_queue.op_completed(())
     }
 
     pub(crate) fn switch_workspaces(