Avoid cloning language server names in Document::save

`language_servers` is a `HashMap<String, Arc<Client>>` so the clients
are cheap to clone but not the language server names. The names are
unused in the save future so we can avoid the unnecessary clones by
looking only at the `HashMap::values`.

Author: the-mikedavis

helix-view/src/document.rs

@@ -981,7 +981,7 @@ impl Document {
         };
 
         let identifier = self.path().map(|_| self.identifier());
-        let language_servers = self.language_servers.clone();
+        let language_servers: Vec<_> = self.language_servers.values().cloned().collect();
 
         // mark changes up to now as saved
         let current_rev = self.get_current_revision();
@@ -1125,7 +1125,7 @@ impl Document {
                 text: text.clone(),
             };
 
-            for (_, language_server) in language_servers {
+            for language_server in language_servers {
                 if !language_server.is_initialized() {
                     continue;
                 }