microsoft / microsoft/TypeScript
CFA should consider branch flags
Nessuno ha ancora preso questa issue.
- Lingua principale
- Go
- Stelle
- 111k
- Fork
- 14.3k
- Merge medio
- 1g 19h
- PR unite (30g)
- 117
Descrizione
This is a suggestion to improve the CFA. It would be good if TypeScript considered branch flags when analysing code paths.
function f() {
let text: string;
let hasTakenVerySpecificBranch = false;
if (0.5 < Math.random()) {
if (0.5 < Math.random()) {
if (0.5 < Math.random()) {
hasTakenVerySpecificBranch = true;
text = '';
}
}
}
if (!hasTakenVerySpecificBranch) {
text = ''
}
console.log(text); // Error 'text' is used before assigned.
}
We assign text on all possible code paths above. And it shouldn't error above.
A branch flag is simply a boolean variable that is considered when doing the control flow analysis.
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 riproducendo l’esempio TypeScript e confermando l’errore attuale di definite assignment. Leggi l’implementazione dell’analisi del control-flow del compilatore e i test esistenti per definite assignment e boolean narrowing. Il lavoro è completato quando le condizioni con branch flags vengono incorporate senza regressioni nel comportamento di control-flow correlato, con test che coprono l’esempio.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- typescript
- Ambito
- compilers
- Tipo di issue
- Funzionalità
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 30/100