Missing warning about removing mut qualifier?

[djc]

During work on Cargo, I found a seeming bug where the compiler doesn't warn about an unnecessary mut qualifier. This version does not warn:

fn bar<'a>(foo: &'a String) -> bool {
    foo == "foo"
}

fn main() {
    let res = {
        let mut foo = String::from("foo");
        bar(&foo)
    };
    println!("{:?}", res);
}

This version does warn:

fn bar<'a>(foo: &'a String) -> bool {
    foo == "foo"
}

fn main() {
    let mut foo = String::from("foo");
    let res = bar(&foo);
    println!("{:?}", res);
}

[estebank]

It warns now:

warning: variable does not need to be mutable
 --> src/main.rs:8:13
  |
8 |         let mut foo = String::from("foo");
  |             ----^^^
  |             |
  |             help: remove this `mut`
  |
  = note: #[warn(unused_mut)] on by default)

[djc]

Yeah, I suppose this has been fixed with NLL.