__decorate helper should not use `this` when targeting modules

TypeScript Version: 3.7.x-dev.201xxxxx

Search Terms: __decorate this module

Code

class Foo {
  @property() x = 1;
}

Expected behavior:
Top-level this reference is not emitted.

Actual behavior:

The __decrate variable is declared like:

var __decorate = (this && this.__decorate) || function (decorators, target, key, desc) {

Since top-level this is always undefined in modules, (this && this.__decorate) || can be omitted. It's only a few bytes, but the presence of top-level this also causes warnings in other downstream tools like Rollup.

A few other things could be fixed for an ES6+ helper, btw:

• Use const instead of var (allows for some VM optimizations)
• Remove Reflect.decorate
• Don't use arguments

Playground Link: https://www.typescriptlang.org/play/?ts=Nightly#code/MYGwhgzhAEBiD29oG8BQ1oAEAOAne2AprgC4CeAFAJTQAe0AvNAIwDcqAvkA

Related Issues: None?

Tái hiện ví dụ từ TypeScript Playground được liên kết bằng phiên bản 3.7.x-dev đã báo cáo hoặc compiler hiện tại, và kiểm tra helper __decorate được phát ra. Truy tìm entry point của compiler phát ra helper này; công việc được hoàn tất khi output của module không còn bao gồm tham chiếu this ở cấp cao nhất, trong khi hành vi của decorator vẫn được giữ nguyên.