---

yaml
---
r: 160127
b: refs/heads/issue-18208-method-dispatch-3-quick-reject
c: 9ba6a6e
h: refs/heads/master
i:
  160125: 0d26178
  160123: 55ccdc9
  160119: f9d919e
  160111: 3904dd5
  160095: ddafd2e
  160063: 3e5e91f
  159999: 13d4cc4
v: v3

Author: nikomatsakis

[refs]

@@ -28,4 +28,4 @@ refs/tags/0.11.0: e1247cb1d0d681be034adb4b558b5a0c0d5720f9
 refs/tags/0.12.0: f0c419429ef30723ceaf6b42f9b5a2aeb5d2e2d1
 refs/heads/issue-18208-method-dispatch-2: 9e1eae4fb9b6527315b4441cf8a0f5ca911d1671
 refs/heads/automation-fail: 1bf06495443584539b958873e04cc2f864ab10e4
-refs/heads/issue-18208-method-dispatch-3-quick-reject: 715129786620b3fce1b8efccb98ddab992287ff0
+refs/heads/issue-18208-method-dispatch-3-quick-reject: 9ba6a6e74b8ab986e3f2246926e3aba369dea0e6

branches/issue-18208-method-dispatch-3-quick-reject/src/librustc/middle/typeck/check/method/probe.rs

@@ -827,6 +827,32 @@ impl<'a,'tcx> ProbeContext<'a,'tcx> {
                method.fty.sig.inputs[0].repr(self.tcx()),
                substs.repr(self.tcx()));
 
+        // It is possible for type parameters or early-bound lifetimes
+        // to appear in the signature of `self`. The substitutions we
+        // are given do not include type/lifetime parameters for the
+        // method yet. So create fresh variables here for those too,
+        // if there are any.
+        assert_eq!(substs.types.len(subst::FnSpace), 0);
+        assert_eq!(substs.regions().len(subst::FnSpace), 0);
+        let mut substs = substs;
+        let placeholder;
+        if
+            !method.generics.types.is_empty_in(subst::FnSpace) ||
+            !method.generics.regions.is_empty_in(subst::FnSpace)
+        {
+            let method_types =
+                self.infcx().next_ty_vars(
+                    method.generics.types.len(subst::FnSpace));
+
+            let method_regions =
+                self.infcx().region_vars_for_defs(
+                    self.span,
+                    method.generics.regions.get_slice(subst::FnSpace));
+
+            placeholder = (*substs).clone().with_method(method_types, method_regions);
+            substs = &placeholder;
+        }
+
         let xform_self_ty = method.fty.sig.inputs[0].subst(self.tcx(), substs);
         self.infcx().replace_late_bound_regions_with_fresh_var(method.fty.sig.binder_id,
                                                                self.span,

branches/issue-18208-method-dispatch-3-quick-reject/src/test/run-pass/method-early-bound-lifetimes-on-self.rs

@@ -0,0 +1,34 @@
+// Copyright 2012 The Rust Project Developers. See the COPYRIGHT
+// file at the top-level directory of this distribution and at
+// http://rust-lang.org/COPYRIGHT.
+//
+// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
+// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
+// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
+// option. This file may not be copied, modified, or distributed
+// except according to those terms.
+
+// Check that we successfully handle methods where the `self` type has
+// an early-bound lifetime. Issue #18208.
+
+#![allow(dead_code)]
+
+struct Cursor<'a>;
+
+trait CursorNavigator {
+    fn init_cursor<'a, 'b:'a>(&'a self, cursor: &mut Cursor<'b>) -> bool;
+}
+
+struct SimpleNavigator;
+
+impl CursorNavigator for SimpleNavigator {
+    fn init_cursor<'a, 'b: 'a>(&'a self, _cursor: &mut Cursor<'b>) -> bool {
+        false
+    }
+}
+
+fn main() {
+    let mut c = Cursor;
+    let n = SimpleNavigator;
+    n.init_cursor(&mut c);
+}