CECP engine forfeits on claim of illegal move

alwey · ilaripih · commit 2fb5d66fbc44 · 2018-08-01T20:02:25.000+03:00
As long as cutechess has an arbiter function, illegal moves are not (read: should not be) transmitted to engines. So a claim of an illegal move should not be valid. At present, without adjudication the engine waits and finally forfeits on time.
diff --git a/projects/lib/src/xboardengine.cpp b/projects/lib/src/xboardengine.cpp
@@ -759,6 +759,13 @@ void XboardEngine::parseLine(const QString& line)
 			pos += rx.matchedLength();
 		}
 	}
+	else if (command.startsWith("Illegal"))
+	{
+		forfeit(Chess::Result::Adjudication,
+			tr("%1 claims illegal %2")
+			.arg(this->side().toString())
+			.arg(args));
+	}
 	else if (command == "Error")
 	{
 		// If the engine complains about an unknown result command,

Original file line number	Diff line number	Diff line change
`@@ -759,6 +759,13 @@ void XboardEngine::parseLine(const QString& line)`
`759`	`759`	`pos += rx.matchedLength();`
`760`	`760`	`}`
`761`	`761`	`}`
	`762`	`+ else if (command.startsWith("Illegal"))`
	`763`	`+ {`
	`764`	`+ forfeit(Chess::Result::Adjudication,`
	`765`	`+ tr("%1 claims illegal %2")`
	`766`	`+ .arg(this->side().toString())`
	`767`	`+ .arg(args));`
	`768`	`+ }`
`762`	`769`	`else if (command == "Error")`
`763`	`770`	`{`
`764`	`771`	`// If the engine complains about an unknown result command,`