You can never call your own fn borrow(&mut self, ...) method if you use std::borrow::Borrow;

[bombless]

https://play.rust-lang.org/?gist=c36d8c617e66862b7bcaee05752ad313&version=stable&mode=debug&edition=2015

fn main() {
    use std::borrow::Borrow;
    struct S;
    impl S { fn borrow(&mut self, _: ()) {} }
    let mut s = S;
    s.borrow(())
}

(Just noticed that this might be the intended behaviour, since std::clone::Clone acts the same. But still, the error message here might need to be improved)

[bombless]

it looks fishy since if you change the code to impl S { fn borrow(&self, _: ()) {} } it compiles, which doesn't look logical to me.

[csmoe]

cc https://doc.rust-lang.org/reference/expressions/method-call-expr.html#method-call-expressions
&self methods are looked up first, the trait method is found before the struct's &mut self method is found.

[ishitatsuyuki]

This is how name resolution works, and yeah you need to use UFCS if you shadow such names.

Labeling this as a diagnostics feature request.

[ExpHP]

&self methods are looked up first, the trait method is found before the struct's &mut self method is found.

But... why?

Is this something that could perhaps one day change with a crater run? To me it looks like a terrible corner case with no useful outcome, and acts as little more than an exception to the "rule" that inherent methods come before trait methods. (a rule that is actually useful!)

[steveklabnik]

The error message:

error[E0061]: this function takes 0 parameters but 1 parameter was supplied
 --> src/main.rs:6:7
  |
6 |     s.borrow(())
  |       ^^^^^^ expected 0 parameters

error[E0308]: mismatched types
 --> src/main.rs:6:5
  |
1 | fn main() {
  |           - expected `()` because of default return type
...
6 |     s.borrow(())
  |     ^^^^^^^^^^^^- help: try adding a semicolon: `;`
  |     |
  |     expected (), found &main::S
  |
  = note: expected type `()`
             found type `&main::S`

[estebank]

This should suggest to alternatively write S::borrow(&mut s, ());.

[estebank]

Slight change. It now points at the definition of Borrow::borrow, but it still doesn't mention the inherent impl fn.

error[E0061]: this method takes 0 arguments but 1 argument was supplied
 --> src/main.rs:6:7
  |
6 |     s.borrow(())
  |       ^^^^^^ --
  |              |
  |              unexpected argument of type `()`
  |              help: remove the extra argument
  |
note: method defined here
 --> /rustc/fee5518cdd4435c60a57fe3bb734fc1a14abeb7a/library/core/src/borrow.rs:178:8

error[E0308]: mismatched types
 --> src/main.rs:6:5
  |
1 | fn main() {
  |          - expected `()` because of default return type
...
6 |     s.borrow(())
  |     ^^^^^^^^^^^^- help: consider using a semicolon here: `;`
  |     |
  |     expected `()`, found `&S`