microsoft/Terminal

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

Aperta

#11.545 aperta il 20 ott 2021

Vedi su GitHubGuida contributor
 (7 commenti) (0 reazioni) (0 assegnatari)C++ (9275 fork)batch import
Area-SettingsHelp WantedIssue-BugProduct-TerminalResolution-External

Metriche repository

Star
 (103.173 stelle)
Metriche merge PR
 (Merge medio 27g 19h) (24 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Indaga la logica di parsing JSON nel codice sorgente di Windows Terminal, probabilmente nella directory 'src'. Cerca come vengono gestite le virgole finali e i commenti. L'issue #11545 discute un conflitto tra il consentire virgole finali e commenti. Verifica se esistono test esistenti per il parsing JSON. Rivedi i commenti nell'issue per potenziali soluzioni. La correzione potrebbe comportare la modifica dell'ordine di controllo per le virgole finali e i commenti nel parser.
Tech stack
cpp
Dominio
desktop
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.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75