@@ -440,30 +440,29 @@ impl<'a, 'tcx> FnCtxt<'a, 'tcx> {
440
440
call_expr : & hir:: Expr < ' tcx > ,
441
441
) {
442
442
// Next, let's construct the error
443
- let ( error_span, full_call_span, ctor_of) = match & call_expr. kind {
443
+ let ( error_span, full_call_span, ctor_of, is_method ) = match & call_expr. kind {
444
444
hir:: ExprKind :: Call (
445
445
hir:: Expr { hir_id, span, kind : hir:: ExprKind :: Path ( qpath) , .. } ,
446
446
_,
447
447
) => {
448
448
if let Res :: Def ( DefKind :: Ctor ( of, _) , _) =
449
449
self . typeck_results . borrow ( ) . qpath_res ( qpath, * hir_id)
450
450
{
451
- ( call_span, * span, Some ( of) )
451
+ ( call_span, * span, Some ( of) , false )
452
452
} else {
453
- ( call_span, * span, None )
453
+ ( call_span, * span, None , false )
454
454
}
455
455
}
456
- hir:: ExprKind :: Call ( hir:: Expr { span, .. } , _) => ( call_span, * span, None ) ,
456
+ hir:: ExprKind :: Call ( hir:: Expr { span, .. } , _) => ( call_span, * span, None , false ) ,
457
457
hir:: ExprKind :: MethodCall ( path_segment, _, span) => {
458
458
let ident_span = path_segment. ident . span ;
459
459
let ident_span = if let Some ( args) = path_segment. args {
460
460
ident_span. with_hi ( args. span_ext . hi ( ) )
461
461
} else {
462
462
ident_span
463
463
} ;
464
- (
465
- * span, ident_span, None , // methods are never ctors
466
- )
464
+ // methods are never ctors
465
+ ( * span, ident_span, None , true )
467
466
}
468
467
k => span_bug ! ( call_span, "checking argument types on a non-call: `{:?}`" , k) ,
469
468
} ;
@@ -659,7 +658,7 @@ impl<'a, 'tcx> FnCtxt<'a, 'tcx> {
659
658
Applicability :: MachineApplicable ,
660
659
) ;
661
660
} ;
662
- self . label_fn_like ( & mut err, fn_def_id, callee_ty) ;
661
+ self . label_fn_like ( & mut err, fn_def_id, callee_ty, Some ( mismatch_idx ) , is_method ) ;
663
662
err. emit ( ) ;
664
663
return ;
665
664
}
@@ -701,16 +700,14 @@ impl<'a, 'tcx> FnCtxt<'a, 'tcx> {
701
700
}
702
701
703
702
errors. drain_filter ( |error| {
704
- let Error :: Invalid ( provided_idx, expected_idx, Compatibility :: Incompatible ( error ) ) = error else { return false } ;
703
+ let Error :: Invalid ( provided_idx, expected_idx, Compatibility :: Incompatible ( Some ( e ) ) ) = error else { return false } ;
705
704
let ( provided_ty, provided_span) = provided_arg_tys[ * provided_idx] ;
706
705
let ( expected_ty, _) = formal_and_expected_inputs[ * expected_idx] ;
707
706
let cause = & self . misc ( provided_span) ;
708
707
let trace = TypeTrace :: types ( cause, true , expected_ty, provided_ty) ;
709
- if let Some ( e) = error {
710
- if !matches ! ( trace. cause. as_failure_code( e) , FailureCode :: Error0308 ( _) ) {
711
- self . report_and_explain_type_error ( trace, e) . emit ( ) ;
712
- return true ;
713
- }
708
+ if !matches ! ( trace. cause. as_failure_code( e) , FailureCode :: Error0308 ( _) ) {
709
+ self . report_and_explain_type_error ( trace, e) . emit ( ) ;
710
+ return true ;
714
711
}
715
712
false
716
713
} ) ;
@@ -749,7 +746,7 @@ impl<'a, 'tcx> FnCtxt<'a, 'tcx> {
749
746
format ! ( "arguments to this {} are incorrect" , call_name) ,
750
747
) ;
751
748
// Call out where the function is defined
752
- self . label_fn_like ( & mut err, fn_def_id, callee_ty) ;
749
+ self . label_fn_like ( & mut err, fn_def_id, callee_ty, Some ( expected_idx . as_usize ( ) ) , is_method ) ;
753
750
err. emit ( ) ;
754
751
return ;
755
752
}
@@ -1031,7 +1028,7 @@ impl<'a, 'tcx> FnCtxt<'a, 'tcx> {
1031
1028
}
1032
1029
1033
1030
// Call out where the function is defined
1034
- self . label_fn_like ( & mut err, fn_def_id, callee_ty) ;
1031
+ self . label_fn_like ( & mut err, fn_def_id, callee_ty, None , is_method ) ;
1035
1032
1036
1033
// And add a suggestion block for all of the parameters
1037
1034
let suggestion_text = match suggestion_text {
@@ -1781,6 +1778,9 @@ impl<'a, 'tcx> FnCtxt<'a, 'tcx> {
1781
1778
err : & mut Diagnostic ,
1782
1779
callable_def_id : Option < DefId > ,
1783
1780
callee_ty : Option < Ty < ' tcx > > ,
1781
+ // A specific argument should be labeled, instead of all of them
1782
+ expected_idx : Option < usize > ,
1783
+ is_method : bool ,
1784
1784
) {
1785
1785
let Some ( mut def_id) = callable_def_id else {
1786
1786
return ;
@@ -1881,10 +1881,17 @@ impl<'a, 'tcx> FnCtxt<'a, 'tcx> {
1881
1881
. get_if_local ( def_id)
1882
1882
. and_then ( |node| node. body_id ( ) )
1883
1883
. into_iter ( )
1884
- . flat_map ( |id| self . tcx . hir ( ) . body ( id) . params ) ;
1884
+ . flat_map ( |id| self . tcx . hir ( ) . body ( id) . params )
1885
+ . skip ( if is_method { 1 } else { 0 } ) ;
1885
1886
1886
- for param in params {
1887
- spans. push_span_label ( param. span , "" ) ;
1887
+ for ( idx, param) in params. into_iter ( ) . enumerate ( ) {
1888
+ if let Some ( expected_idx) = expected_idx {
1889
+ if idx == expected_idx {
1890
+ spans. push_span_label ( param. span , "" ) ;
1891
+ }
1892
+ } else {
1893
+ spans. push_span_label ( param. span , "" ) ;
1894
+ }
1888
1895
}
1889
1896
1890
1897
let def_kind = self . tcx . def_kind ( def_id) ;
0 commit comments