Microsoft/TypeScript

Middle operand in comma operator list not properly checked for side-effect-freeness

Aperta

#33.646 aperta il 27 set 2019

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)TypeScript (13.395 fork)batch import
BugDomain: Error MessagesHelp Wanted

Metriche repository

Star
 (108.860 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

TypeScript Version: master at f304b81fa54285f Search Terms: comma side effects

Code

declare const obj: any;
console.log(`${JSON.stringify(obj), undefined, 2}`);

Expected behavior: error TS2695: Left side of comma operator is unused and has no side effects.

Actual behavior: No errors

Playground Link: http://www.typescriptlang.org/play/#code/CYUwxgNghgTiAEYD2A7AzgF3kgRgKwC54oUBPAbgChl0kIQA6CJAcwAoADAEgG8ApAMoB5AHINMMAJYoWkgGak2uPAEoANPACuKUHOkhgGgEwBfDiqpA

Related Issues: #10814

Guida contributor

Direzione di ricerca
Ispeziona la logica di controllo degli effetti collaterali dell'operatore virgola nel checker TypeScript (checker.ts). Il problema segnala che l'operando centrale in una lista di virgole non viene controllato per l'assenza di effetti collaterali. Cerca dove vengono validate le espressioni con operatore virgola e assicurati che tutti gli operandi tranne l'ultimo vengano controllati.
Tech stack
typescript
Dominio
backenddeveloper experience
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
TypeScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70