Auto merge of #87686 - matthiaskrgr:clippy_august_21_perf, r=jackh726
clippy::perf fixes
This commit is contained in:
commit
d77dcbc599
3 changed files with 7 additions and 7 deletions
|
@ -265,7 +265,7 @@ impl<'a, 'tcx> FnCtxt<'a, 'tcx> {
|
|||
// Get the number of generics the self type has (if an Adt) unless we can determine that
|
||||
// the user has written the self type with generics already which we (naively) do by looking
|
||||
// for a "<" in `self_ty_name`.
|
||||
Adt(def, _) if !self_ty_name.contains("<") => self.tcx.generics_of(def.did).count(),
|
||||
Adt(def, _) if !self_ty_name.contains('<') => self.tcx.generics_of(def.did).count(),
|
||||
_ => 0,
|
||||
};
|
||||
let self_ty_generics = if self_ty_generics_count > 0 {
|
||||
|
|
|
@ -222,8 +222,8 @@ impl ItemLikeVisitor<'v> for InherentOverlapChecker<'tcx> {
|
|||
let id_to_set = *ids.iter().min().unwrap();
|
||||
|
||||
// Sort the id list so that the algorithm is deterministic
|
||||
let mut ids = ids.into_iter().collect::<SmallVec<[_; 8]>>();
|
||||
ids.sort();
|
||||
let mut ids = ids.into_iter().collect::<SmallVec<[usize; 8]>>();
|
||||
ids.sort_unstable();
|
||||
|
||||
let mut region = connected_regions.remove(&id_to_set).unwrap();
|
||||
region.idents.extend_from_slice(&idents_to_add);
|
||||
|
@ -266,8 +266,8 @@ impl ItemLikeVisitor<'v> for InherentOverlapChecker<'tcx> {
|
|||
// for each pair of impl blocks in the same connected region.
|
||||
for (_id, region) in connected_regions.into_iter() {
|
||||
let mut impl_blocks =
|
||||
region.impl_blocks.into_iter().collect::<SmallVec<[_; 8]>>();
|
||||
impl_blocks.sort();
|
||||
region.impl_blocks.into_iter().collect::<SmallVec<[usize; 8]>>();
|
||||
impl_blocks.sort_unstable();
|
||||
for (i, &impl1_items_idx) in impl_blocks.iter().enumerate() {
|
||||
let &(&impl1_def_id, impl_items1) = &impls_items[impl1_items_idx];
|
||||
for &impl2_items_idx in impl_blocks[(i + 1)..].iter() {
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue