andersfylling / andersfylling/uci

Incorrect parsing when specifying multiple subcommands

Ouverte
#1 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
Langage dominant
C++
Étoiles
3
Forks
1
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

When parsing the UCI command `go wtime 8153 btime 8080 winc 80 binc 80`, the output is `{"wtime": "8153 btime 8080 winc 80 binc 80"}`, instead of `{"binc": "80", "btime": "8080", "winc": "80", "wtime": "8153"}`.

Looking at [`Parser.cpp`](https://github.com/andersfylling/uci/blob/master/uci_code/src/Parser.cpp), we see that the parser doesn't support multiple subcommands, since it doesn't exit the `while (request >> nextWord)` loop until the entire line has been exhausted.

This is not caught by the unit tests, since it never tests the case of multiple subcommands, however this is a necessary feature for compatibility with UCI GUIs.

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.