Is there anything fundamentally wrong with this code?

[aberba via Digitalmars-d-learn]

On Friday, 3 February 2017 at 22:34:31 UTC, Ali Çehreli wrote:
> On 02/03/2017 11:43 AM, WhatMeWorry wrote:
>> On Friday, 3 February 2017 at 18:37:15 UTC, Johan Engelen 
>> wrote:
>>> On Friday, 3 February 2017 at 17:20:43 UTC, WhatMeWorry wrote:
>>>>         [...]
>...
> Another related one is assigning to a parameter usually in the 
> constructor:
>
> struct S {
>     int i;
>
>     this(int i) {
>         i = i;    // meant this.i = i
>     }
> }
>
> Ali

Most D devs ignore the "this" in their code and that has 
influenced me to do that often. Is it no prone to bugs?