Module level variable shadowing

[Kapps via Digitalmars-d]

On Saturday, 28 June 2014 at 05:13:16 UTC, H. S. Teoh via 
Digitalmars-d wrote:
>
> Actually, this particular use case is very bad. It's just 
> inviting
> typos, for example, if you mistyped "int a" as "int s", then 
> you get:
>
> 	struct Foo {
> 		int a;
> 		this(int s) {
> 			this.a = a; // oops, now it means this.a = this.a
> 		}
> 	}
>

While I'd much prefer a syntax to handle this automatically, like:

struct Foo {
     int a;
     this(this.a) { }
}

I prefer having the variable names match what they're assigning 
since it makes it clearer. Some other benefits include being able 
to get documentation for free. When I was making my own IDE 
plugin a year ago, one of the features I liked was that if a 
parameter in the constructor had the same name as a 
field/property in the class, documentation would be shown for 
said field/property as well as for the parameter.

It's just a matter of personal preference. In theory your 
compiler should tell you if you're making a useless assignment 
like the one in the example.