auto merge of #5096 : luqmana/rust/spell, r=catamorphism

Address #2281


```
-> % cat foo.rs
fn foo() -> int {
    let bar = 10;

    bad
}
```

```
-> % rustc foo.rs
foo.rs:4:4: 4:7 error: unresolved name: `bad`. Did you mean: `bar`?
foo.rs:4     bad
             ^~~
error: aborting due to previous error
```
This commit is contained in:
bors 2013-02-26 19:03:40 -08:00
commit 28b50a4892
10 changed files with 92 additions and 11 deletions

View file

@ -590,6 +590,40 @@ pub pure fn split_str_nonempty(s: &a/str, sep: &b/str) -> ~[~str] {
result
}
/// Levenshtein Distance between two strings
pub fn levdistance(s: &str, t: &str) -> uint {
let slen = str::len(s);
let tlen = str::len(t);
if slen == 0 { return tlen; }
if tlen == 0 { return slen; }
let mut dcol = vec::from_fn(tlen + 1, |x| x);
for str::each_chari(s) |i, sc| {
let mut current = i;
dcol[0] = current + 1;
for str::each_chari(t) |j, tc| {
let mut next = dcol[j + 1];
if sc == tc {
dcol[j + 1] = current;
} else {
dcol[j + 1] = ::cmp::min(current, next);
dcol[j + 1] = ::cmp::min(dcol[j + 1], dcol[j]) + 1;
}
current = next;
}
}
return dcol[tlen];
}
/**
* Splits a string into a vector of the substrings separated by LF ('\n')
*/

View file

@ -4816,6 +4816,42 @@ pub impl Resolver {
}
}
fn find_best_match_for_name(@mut self, name: &str) -> Option<~str> {
let mut maybes: ~[~str] = ~[];
let mut values: ~[uint] = ~[];
let mut j = self.value_ribs.len();
while j != 0 {
j -= 1;
let rib = self.value_ribs.get_elt(j);
for rib.bindings.each_entry |e| {
vec::push(&mut maybes, copy *self.session.str_of(e.key));
vec::push(&mut values, uint::max_value);
}
}
let mut smallest = 0;
for vec::eachi(maybes) |i, &other| {
values[i] = str::levdistance(name, other);
if values[i] <= values[smallest] {
smallest = i;
}
}
if vec::len(values) > 0 &&
values[smallest] != uint::max_value &&
values[smallest] < str::len(name) + 2 &&
maybes[smallest] != name.to_owned() {
Some(vec::swap_remove(&mut maybes, smallest))
} else {
None
}
}
fn name_exists_in_scope_struct(@mut self, name: &str) -> bool {
let mut i = self.type_ribs.len();
while i != 0 {
@ -4882,9 +4918,20 @@ pub impl Resolver {
wrong_name));
}
else {
self.session.span_err(expr.span,
fmt!("unresolved name: %s",
match self.find_best_match_for_name(wrong_name) {
Some(m) => {
self.session.span_err(expr.span,
fmt!("unresolved name: `%s`. \
Did you mean: `%s`?",
wrong_name, m));
}
None => {
self.session.span_err(expr.span,
fmt!("unresolved name: `%s`.",
wrong_name));
}
}
}
}
}

View file

@ -16,6 +16,6 @@ fn my_fail() -> ! { fail!(); }
fn main() {
match true { false => { my_fail(); } true => { } }
log(debug, x); //~ ERROR unresolved name: x
log(debug, x); //~ ERROR unresolved name: `x`.
let x: int;
}

View file

@ -8,7 +8,7 @@
// option. This file may not be copied, modified, or distributed
// except according to those terms.
// error-pattern: unresolved name: m1::a
// error-pattern: unresolved name: `m1::a`. Did you mean: `args`?
mod m1 {}

View file

@ -8,7 +8,7 @@
// option. This file may not be copied, modified, or distributed
// except according to those terms.
// error-pattern: unresolved name: m1::a
// error-pattern: unresolved name: `m1::a`. Did you mean: `args`?
mod m1 {
pub mod a {}

View file

@ -1,3 +1,3 @@
// error-pattern: unresolved name: this_does_nothing_what_the
// error-pattern: unresolved name: `this_does_nothing_what_the`.
fn main() { debug!("doing"); this_does_nothing_what_the; debug!("boing"); }

View file

@ -9,5 +9,5 @@
// except according to those terms.
fn main() {
log(error, x); //~ ERROR unresolved name: x
log(error, x); //~ ERROR unresolved name: `x`.
}

View file

@ -19,7 +19,7 @@ fn siphash(k0 : u64) {
impl siphash {
fn reset(&mut self) {
self.v0 = k0 ^ 0x736f6d6570736575; //~ ERROR attempted dynamic environment-capture
//~^ ERROR unresolved name: k0
//~^ ERROR unresolved name: `k0`.
}
}
}

View file

@ -31,9 +31,9 @@ fn siphash(k0 : u64, k1 : u64) -> siphash {
impl siphash for sipstate {
fn reset() {
self.v0 = k0 ^ 0x736f6d6570736575; //~ ERROR attempted dynamic environment-capture
//~^ ERROR unresolved name: k0
//~^ ERROR unresolved name: `k0`.
self.v1 = k1 ^ 0x646f72616e646f6d; //~ ERROR attempted dynamic environment-capture
//~^ ERROR unresolved name: k1
//~^ ERROR unresolved name: `k1`.
}
fn result() -> u64 { return mk_result(self); }
}

View file

@ -23,7 +23,7 @@ fn siphash(k0 : u64) -> siphash {
impl siphash for sipstate {
fn reset() {
self.v0 = k0 ^ 0x736f6d6570736575; //~ ERROR attempted dynamic environment-capture
//~^ ERROR unresolved name: k0
//~^ ERROR unresolved name: `k0`.
}
}
fail!();