Microsoft/TypeScript

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

Aberta

#33.646 aberto em 27 de set. de 2019

Ver no GitHubGuia do colaborador
 (0 comentário) (0 reação) (0 responsável)TypeScript (13.395 forks)batch import
BugDomain: Error MessagesHelp Wanted

Métricas do repositório

Stars
 (108.860 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

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

Guia do colaborador

Direção de pesquisa
Inspecione a lógica de verificação de efeitos colaterais do operador vírgula no verificador TypeScript (checker.ts). O problema relata que o operando do meio em uma lista de vírgulas não é verificado quanto à ausência de efeitos colaterais. Procure onde as expressões de operador vírgula são validadas e garanta que todos os operandos, exceto o último, sejam verificados.
Pilha de tecnologia
typescript
Domain
backenddeveloper experience
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
TypeScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70