microsoft/Terminal

JSON fails to parse with a trailing comment at the end of an array

Open

#11.545 aberto em 20 de out. de 2021

Ver no GitHub
 (7 comments) (0 reactions) (0 assignees)C++ (3.212 forks)batch import
Area-SettingsHelp WantedIssue-BugProduct-TerminalResolution-External

Métricas do repositório

Stars
 (35.764 stars)
Métricas de merge de PR
 (Mesclagem média 27d 19h) (24 fundiu PRs em 30d)

Description

Windows Terminal version (or Windows build number)

1.10.2714.0

Other Software

No response

Steps to reproduce

In the settings file, put a comment at the end of an object with a comma after it.

This fails:

"actions": [
  {},
  // ...
],

Dropping either the , or the comment makes it work.

Expected Behavior

Should be parsed as usual since both terminating commas and comments are allowed, or forbid trailing commas (assuming that comments are a desired feature).

Actual Behavior

Settings could not be loaded from file. Check for syntax errors, including trailing commas.

Guia do colaborador

Direção de pesquisa
Investigue a lógica de análise JSON no código fonte do Windows Terminal, provavelmente no diretório 'src'. Procure como as vírgulas finais e os comentários são tratados. A issue #11545 discute um conflito entre permitir vírgulas finais e comentários. Verifique se existem testes existentes para análise JSON. Revise os comentários na issue para possíveis soluções. A correção pode envolver ajustar a ordem de verificação de vírgulas finais e comentários no analisador.
Pilha de tecnologia
cpp
Domain
desktop
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.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75