Allow empty return statements on setters

[gjuchault]

TypeScript Version:
1.8.10

Code

class Foo {
  private _value: string = '';
  set value(newV: string): void {
    if (newV === this._value) {
      return;
    }

    // ...
  }
}

Expected behavior:

No errors.

Actual behavior:

A 'set' accessor cannot have a return type annotation..

Thanks :)

[aluanhaddad]

Is the issue of the Void annotation, or the return statement I believe it's the former.

[gjuchault]

The error is that TypeScript puts an error when there is a return statement on a setter; when it can makes sense to stop the setter becasue of validation or whatever

[RyanCavanaugh]

The issue is the return type annotation, not the return statement. Expressionless return statements are legal. Return type annotations are not.

[gjuchault]

@RyanCavanaugh Ok I'm not really sure what's the difference between the two. Is the : void non-sense on a setter ?

[RyanCavanaugh]

Yes. The returned value of a setter is unobservable, so it doesn't make any sense to talk about its return type.