diff options
author | Lukas Wirth <[email protected]> | 2020-12-13 10:34:44 +0000 |
---|---|---|
committer | Lukas Wirth <[email protected]> | 2020-12-13 10:34:44 +0000 |
commit | ae8a8020857080ea527cc1d0e57a592d6e587a64 (patch) | |
tree | 81bc716b34669495f3a30a2624b3f9433dfbace8 /crates/hir_ty | |
parent | 10f633283180d513138d0d969eceba96e3ce9f96 (diff) |
Ignore lifetime params in substitutions
Diffstat (limited to 'crates/hir_ty')
-rw-r--r-- | crates/hir_ty/src/infer/expr.rs | 7 | ||||
-rw-r--r-- | crates/hir_ty/src/lower.rs | 8 |
2 files changed, 13 insertions, 2 deletions
diff --git a/crates/hir_ty/src/infer/expr.rs b/crates/hir_ty/src/infer/expr.rs index ca005bc99..2cdce2cef 100644 --- a/crates/hir_ty/src/infer/expr.rs +++ b/crates/hir_ty/src/infer/expr.rs | |||
@@ -856,7 +856,12 @@ impl<'a> InferenceContext<'a> { | |||
856 | // handle provided type arguments | 856 | // handle provided type arguments |
857 | if let Some(generic_args) = generic_args { | 857 | if let Some(generic_args) = generic_args { |
858 | // if args are provided, it should be all of them, but we can't rely on that | 858 | // if args are provided, it should be all of them, but we can't rely on that |
859 | for arg in generic_args.args.iter().take(type_params) { | 859 | for arg in generic_args |
860 | .args | ||
861 | .iter() | ||
862 | .filter(|arg| matches!(arg, GenericArg::Type(_))) | ||
863 | .take(type_params) | ||
864 | { | ||
860 | match arg { | 865 | match arg { |
861 | GenericArg::Type(type_ref) => { | 866 | GenericArg::Type(type_ref) => { |
862 | let ty = self.make_ty(type_ref); | 867 | let ty = self.make_ty(type_ref); |
diff --git a/crates/hir_ty/src/lower.rs b/crates/hir_ty/src/lower.rs index 92f779360..8392cb770 100644 --- a/crates/hir_ty/src/lower.rs +++ b/crates/hir_ty/src/lower.rs | |||
@@ -565,7 +565,13 @@ fn substs_from_path_segment( | |||
565 | if generic_args.has_self_type { self_params + type_params } else { type_params }; | 565 | if generic_args.has_self_type { self_params + type_params } else { type_params }; |
566 | let skip = if generic_args.has_self_type && self_params == 0 { 1 } else { 0 }; | 566 | let skip = if generic_args.has_self_type && self_params == 0 { 1 } else { 0 }; |
567 | // if args are provided, it should be all of them, but we can't rely on that | 567 | // if args are provided, it should be all of them, but we can't rely on that |
568 | for arg in generic_args.args.iter().skip(skip).take(expected_num) { | 568 | for arg in generic_args |
569 | .args | ||
570 | .iter() | ||
571 | .filter(|arg| matches!(arg, GenericArg::Type(_))) | ||
572 | .skip(skip) | ||
573 | .take(expected_num) | ||
574 | { | ||
569 | match arg { | 575 | match arg { |
570 | GenericArg::Type(type_ref) => { | 576 | GenericArg::Type(type_ref) => { |
571 | had_explicit_type_args = true; | 577 | had_explicit_type_args = true; |