andersfylling / andersfylling/uci

Incorrect parsing when specifying multiple subcommands

オープン
#1 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
C++
スター
3
フォーク
1
PR マージ指標
30日以内にマージされた PR はありません

説明

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.

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。