microsoft / microsoft/TypeScript
Make 'new.target' emit more precautions
オープン
まだ誰も着手していません。
Awaiting More Feedback
Suggestion
- 主要言語
- Go
- スター
- 111k
- フォーク
- 14.3k
- 平均マージ
- 2日 4時間
- マージ済み PR(30日)
- 132
説明
I noticed that the polyfill for new.target ...
class Foo {
constructor() {
if (new.target)
alert("Good.");
else
alert("Bad!");
}
}
... outputs to this:
var Foo = (function () {
function Foo() {
var _newTarget = this.constructor;
if (_newTarget)
alert("Good.");
else
alert("Bad!");
}
return Foo;
}());
That seems dangerous to me, since this is always a true condition for older browsers that don't yet support new.target. Does it not make more sense to create an output such as this?:
var Foo = (function () {
function Foo() {
var _newTarget = this && this.constructor !== Window ? this.constructor : void 0;
/* (have to check 'this' also, in case of strict mode) */
if (_newTarget)
alert("Good.");
else
alert("Bad!");
}
return Foo;
}());
The following code fails to work as expected in Chrome v56.0.2924.87 (output is "Good." in call cases):
class $Foo {
constructor() {
if (new.target)
alert("Good.");
else
alert("Bad!");
}
}
type FooConstructor = typeof $Foo;
interface CallableFoo extends FooConstructor { (): $Foo; }
new $Foo(); // ok
var Foo: CallableFoo = <any>Foo;
Foo(); // ok? :/
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
リンクされている TypeScript Playground の再現コードから始め、new $Foo() と Foo() に対して出力される JavaScript を比較します。コンパイラーでの new.target の lowering を追跡し、生成された出力がコンストラクター呼び出しと通常の呼び出しで異なる結果を維持し、両方を truthy として扱わなければ完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- javascript, typescript
- 領域
- compilers
- issue の種類
- バグ
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 42/100