Update with final comments
This commit is contained in:
parent
1c1ce2fbda
commit
b77bb5cb25
6 changed files with 10 additions and 31 deletions
|
@ -756,7 +756,7 @@ impl<'tcx> AutoTraitFinder<'tcx> {
|
|||
// when we started out trying to unify
|
||||
// some inference variables. See the comment above
|
||||
// for more infomration
|
||||
if p.term().skip_binder().ty().has_infer_types() {
|
||||
if p.term().skip_binder().has_infer_types() {
|
||||
if !self.evaluate_nested_obligations(
|
||||
ty,
|
||||
v.into_iter(),
|
||||
|
|
|
@ -1804,11 +1804,11 @@ impl<'a, 'tcx> InferCtxtPrivExt<'a, 'tcx> for InferCtxt<'a, 'tcx> {
|
|||
}
|
||||
ty::PredicateKind::Projection(data) => {
|
||||
let self_ty = data.projection_ty.self_ty();
|
||||
let ty = data.term.ty();
|
||||
let term = data.term;
|
||||
if predicate.references_error() || self.is_tainted_by_errors() {
|
||||
return;
|
||||
}
|
||||
if self_ty.needs_infer() && ty.needs_infer() {
|
||||
if self_ty.needs_infer() && term.needs_infer() {
|
||||
// We do this for the `foo.collect()?` case to produce a suggestion.
|
||||
let mut err = self.emit_inference_failure_err(
|
||||
body_id,
|
||||
|
|
|
@ -1802,7 +1802,7 @@ fn confirm_param_env_candidate<'cx, 'tcx>(
|
|||
Ok(InferOk { value: _, obligations }) => {
|
||||
nested_obligations.extend(obligations);
|
||||
assoc_ty_own_obligations(selcx, obligation, &mut nested_obligations);
|
||||
// FIXME(...): Handle consts here as well? Maybe this progress type should just take
|
||||
// FIXME(associated_const_equality): Handle consts here as well? Maybe this progress type should just take
|
||||
// a term instead.
|
||||
Progress { ty: cache_entry.term.ty().unwrap(), obligations: nested_obligations }
|
||||
}
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue