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`.
This commit is contained in:
Michael Davis
2025-09-23 10:22:49 -04:00
parent a5d0a0e1c2
commit ce351f4b11

View File

@@ -980,7 +980,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();
@@ -1124,7 +1124,7 @@ impl Document {
text: text.clone(),
};
for (_, language_server) in language_servers {
for language_server in language_servers {
if !language_server.is_initialized() {
continue;
}