languages/rust
languages/rust
1
0
Fork 0
mirror of https://github.com/rust-lang/rust synced 2024-10-04 15:50:51 +00:00
Code Issues Packages Projects Releases Wiki Activity

minor: Remove unnecessary clones

Browse source
This commit is contained in:
Lukas Wirth 2021-12-11 15:27:36 +01:00
parent 143a30aa51
commit 03f6d92ae4
1 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
crates/ide_db/src/helpers/import_assets.rs
View file

@ -280,7 +280,7 @@ fn search_for(
.into_iter()
.filter(|import| import.import_path.len() > 1)
.filter(|import| !scope_definitions.contains(&ScopeDef::from(import.item_to_import)))
.sorted_by_key(|import| import.import_path.clone())
.sorted_by(|a, b| a.import_path.cmp(&b.import_path))
.collect()
}
@ -309,9 +309,9 @@ fn path_applicable_imports(
current_crate,
path_candidate.name.clone(),
// FIXME: we could look up assoc items by the input and propose those in completion,
// but that requries more preparation first:
// but that requires more preparation first:
// * store non-trait assoc items in import_map to fully enable this lookup
// * ensure that does not degrade the performance (bencmark it)
// * ensure that does not degrade the performance (benchmark it)
// * write more logic to check for corresponding trait presence requirement (we're unable to flyimport multiple item right now)
// * improve the associated completion item matching and/or scoring to ensure no noisy completions appear
//