microsoft / microsoft/TypeScript
Make 'new.target' emit more precautions
Nessuno ha ancora preso questa issue.
- Lingua principale
- Go
- Stelle
- 111k
- Fork
- 14.4k
- Merge medio
- 1g 19h
- PR unite (30g)
- 117
Descrizione
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? :/
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia con il repro collegato di TypeScript Playground e confronta il JavaScript emesso per new $Foo() e Foo(). Segui il lowering di new.target nel compilatore; il lavoro è concluso quando l'output generato preserva i risultati diversi per le chiamate al costruttore e le chiamate ordinarie, senza trattarle entrambe come truthy.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- javascript, typescript
- Ambito
- compilers
- Tipo di issue
- Bug
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 42/100