Handling UCI protocol and Stockfish / Glaurung behavior

[sockmonkey]

Code: Select all

***************
*** 155,161 ****
        cerr << "\nBench position&#58; " << cnt << '/' << positions.size&#40;) << endl << endl; 
        if &#40;limitType == "perft") 
            totalNodes += perft&#40;pos, maxDepth * OnePly&#41;; 
-       else if (!think&#40;pos, true, false, 0, dummy, dummy, 0, maxDepth, maxNodes, secsPerPos, moves&#41;) 
            break; 
        totalNodes += nodes_searched&#40;); 
    &#125; 
--- 155,161 ----
        cerr << "\nBench position&#58; " << cnt << '/' << positions.size&#40;) << endl << endl; 
        if &#40;limitType == "perft") 
            totalNodes += perft&#40;pos, maxDepth * OnePly&#41;; 
+       else if (!think&#40;pos, false, false, 0, dummy, dummy, 0, maxDepth, maxNodes, secsPerPos, moves&#41;) 
            break; 
        totalNodes += nodes_searched&#40;); 
    &#125; 

It seems correct to me. Perhaps is the patch file, try to save the file with Linux (/n) line endings and not Windows (/cr/lf) and then apply again using -l or --ignore-whitespace option.

OK, it was --ignore-whitespace that I was missing. Thank you!

Jeremy

[pgeorges]

Marco, I managed to apply both patches to Stockfish 1.6.2 but still have problems under Scid : half the time when moving in a game SF does not work as expected. Maybe some code review regarding UCI handling would fix things.
It is possible that the problem comes from Scid, but it usually works well with UCI engines.

Pascal Georges

hmmmm, have you applied this ?

http://www.talkchess.com/forum/viewtopi ... 43&t=31443

Indeed with this patch, it works well. Thanks.

Pascal Georges