Remnants of YAML 1.1 Spec in YAML 1.2

Aperta
#1,198 4 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
3/5
Tempo stimato
1-2 giorni
Idoneità per principianti
38/100
Tipo di issue
Bug
Chiarezza
Abbastanza chiara
Stato di attività
Ferma
Stack tecnologico
cpp, yaml

Direzione di ricerca

Start with the boolean resolution code in src/convert.cpp at the linked lines and compare its accepted spellings with the YAML 1.2 boolean rules cited in the issue. Check the related tests for boolean conversion and update them to cover the expected spellings. Done means YAML 1.2 true/false variants resolve as booleans while the extra YAML 1.1 spellings do not.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Hello!

I believe there may be an error in how the current codebase handles true/false value resolution for YAML 1.2. The lines in question are linked below:
https://github.com/jbeder/yaml-cpp/blob/0e6e28d1a38224fc8172fae0109ea7f673c096db/src/convert.cpp#L49-L55

According to this section of the YAML 1.2 guide:
https://yaml.org/spec/1.2-old/spec.html#id2805071

Only variants on "true"/"false" in the YAML file should resolve to the booleans. However, the above-linked code also resolves three other ways to represent booleans outside of 1.2 specs but was present in 1.1 specs.

Lingua principale
C++
Stelle
6.1k
Fork
2.2k
Merge medio
22h 2m
PR unite (30g)
1

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di jbeder/yaml-cpp

Tutte le issue di jbeder/yaml-cpp

Issue simili

Altre issue su C++

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.