Implement CECP 'setup' command
authorH.G.Muller <hgm@hgm-xboard.(none)>
Sun, 6 Nov 2016 13:29:08 +0000 (14:29 +0100)
committerH.G.Muller <hgm@hgm-xboard.(none)>
Tue, 8 Nov 2016 18:00:21 +0000 (19:00 +0100)
On reception of an "info string variant NAME startpos FEN" from the engine
UCI2WB will now send "setup (-) 8x8+0_fairy FEN" to the GUI, to make
engine-defined variants also work with UCI engines. Correct operation
required sending of an 'isready' command to the engine after setting
the UCI_Variant option (which evokes the 'info string' response),
and stalling processing of GUI commands until the 'readyok' response,
to prevent early processing of a following 'ping' before the engine
got time to submit the reply. This way the 'info string' response
can be optional. When variant ="chess" the 'info string' is ignored,
as such a setting of UCI_Variant only occurs in response to 'new',
and never to 'variant'.

UCI2WB.c

index 9b197bb..2370615 100644 (file)
--- a/UCI2WB.c
+++ b/UCI2WB.c
@@ -276,8 +276,12 @@ Engine2GUI()
        }\r
        else if(!strcmp(command, "info")) {\r
            int d=0, s=0, t=(GetTickCount() - startTime)/10, n=1;\r
-           char *pv;\r
+           char *pv, varName[80];\r
            if(sscanf(line+5, "string times @ %c", &dummy) == 1) { printf("# %s", line+12); continue; }\r
+           if(sscanf(line+5, "string variant %s", varName) == 1) {\r
+               if(strcmp(varName,"chess") && (p = strstr(line+18, " startpos "))) printf("setup (-) 8x8+0_fairy %s", p+10);\r
+               continue;\r
+           }\r
            if(collect && (pv = strstr(line+5, "currmove "))) {\r
                if(p = strstr(line+5, "currmovenumber ")) {\r
                    n = atoi(p+15);\r
@@ -481,7 +485,10 @@ GUI2Engine()
                iniPos[strlen(iniPos)-1] = sm = 0; collect = (computer == ANALYZE);\r
        }\r
        else if(!strcmp(command, "variant")) {\r
-               if(varOpt) fprintf(toE, "setoption name UCI_Variant value %sucinewgame\n", strcmp(line+8, "3check\n") ? line+8 : "threecheck\n");\r
+               if(varOpt) {\r
+                   fprintf(toE, "setoption name UCI_Variant value %sucinewgame\nisready\n", strcmp(line+8, "3check\n") ? line+8 : "threecheck\n");\r
+                   fflush(toE); Sync(PAUSE);\r
+               }\r
                if(!strcmp(line+8, "shogi\n")) size = 9, strcpy(iniPos, "position startpos");\r
                if(!strcmp(line+8, "5x5+5_shogi\n")) size = 5, strcpy(iniPos, "position startpos");\r
                if(!strcmp(line+8, "xiangqi\n")) strcpy(iniPos, "fen rnbakabnr/9/1c5c1/p1p1p1p1p/9/9/P1P1P1P1P/1C5C1/9/RNBAKABNR r");\r