Name shadowing within a single pattern should have a lint (or be an error)

E.g.

``` rust
let (a, a) = (1, 2);
```

is perfectly valid code, but it seems very useless and one would only do it by accident, i.e. we could lint it.

(That said, I imagine almost all examples of this will hit the "unused variable" warning too.)
