Remove unnecessary moves in closures

Author: Xanewok

rls/src/actions/post_build.rs

@@ -232,10 +232,10 @@ pub struct AnalysisQueue {
 impl AnalysisQueue {
     // Create a new queue and start the worker thread.
     pub fn init() -> AnalysisQueue {
-        let queue = Arc::new(Mutex::new(Vec::new()));
+        let queue = Arc::default();
         let worker_thread = thread::spawn({
             let queue = Arc::clone(&queue);
-            move || AnalysisQueue::run_worker_thread(queue)
+            || AnalysisQueue::run_worker_thread(queue)
         })
         .thread()
         .clone();

rls/src/build/cargo.rs

@@ -59,7 +59,7 @@ pub(super) fn cargo(
         let analysis = Arc::clone(&analysis);
         let input_files = Arc::clone(&input_files);
         let out = Arc::clone(&out);
-        move || {
+        || {
             run_cargo(
                 compilation_cx,
                 package_arg,

rls/src/cmd.rs

@@ -369,7 +369,7 @@ fn init() -> Sender<String> {
         Box::new(ChannelMsgReader::new(receiver)),
         PrintlnOutput,
     );
-    thread::spawn(move || LsService::run(service));
+    thread::spawn(|| LsService::run(service));
 
     sender
         .send(